Bug 210447 - lang/qore: Update to 0.8.12
Summary: lang/qore: Update to 0.8.12
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Wen Heping
URL:
Keywords: easy, needs-qa, patch
Depends on:
Blocks:
 
Reported: 2016-06-21 21:40 UTC by estrabd
Modified: 2016-12-30 01:28 UTC (History)
2 users (show)

See Also:


Attachments
svn diff file to update lang/qore (6.33 KB, patch)
2016-06-21 21:40 UTC, estrabd
no flags Details | Diff
Updated svn diff file to update lang/qore (6.49 KB, patch)
2016-07-29 15:58 UTC, estrabd
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description estrabd 2016-06-21 21:40:36 UTC
Created attachment 171657 [details]
svn diff file to update lang/qore

I tested this with both poudriere and portlint, they were in conflict with warnings so I deferred to what worked with poudriere.

Thank you,
Brett
Comment 1 Pawel Pekala freebsd_committer freebsd_triage 2016-07-03 17:44:00 UTC
===> Portlint output:
FATAL: /usr/local/ports/lang/qore/pkg-plist: [90]: installing pkg-config files into lib/pkgconfig.  All pkg-config files must be installed into libdata/pkgconfig for them to be found by pkg-config.
Comment 2 estrabd 2016-07-14 19:16:34 UTC
Pawel, you see the conflict I was getting between portlink and poudriere. I opted to go with poudriere since making the change that portlint suggesting broke the install on poudriere.

Please advise.
Comment 3 Pawel Pekala freebsd_committer freebsd_triage 2016-07-14 20:08:03 UTC
You need to modify this software's build files to install
it's pkg-config pc files into ${PREFIX}/libdata/pkgconfig directory
like portlint advises, note that warning has FATAL status.

In the meanwhile distfile seems to be modified and fails checksum check:

=> Attempting to fetch https://codeload.github.com/qorelanguage/qore/tar.gz/0.8.12?dummy=/qorelanguage-qore-0.8.12_GH0.tar.gz
fetch: https://codeload.github.com/qorelanguage/qore/tar.gz/0.8.12?dummy=/qorelanguage-qore-0.8.12_GH0.tar.gz: size mismatch: expected 2485749, actual 2486064
=> Attempting to fetch http://distcache.FreeBSD.org/ports-distfiles/qorelanguage-qore-0.8.12_GH0.tar.gz
fetch: http://distcache.FreeBSD.org/ports-distfiles/qorelanguage-qore-0.8.12_GH0.tar.gz: Not Found
=> Couldn't fetch it - please try to retrieve this
=> port manually into /hdd/ports/distfiles/ and try again.
*** Error code 1
Comment 4 estrabd 2016-07-28 16:23:14 UTC
Ok, thank you.
Comment 5 estrabd 2016-07-29 15:58:36 UTC
Created attachment 173097 [details]
Updated svn diff file to update lang/qore

I updated this diff to reflect the suggestions in Comment #3 below.

I tested this with both poudriere and portlint, they were in conflict with warnings so I deferred to what made portlint happy.
Comment 6 estrabd 2016-07-29 15:59:07 UTC
Comment on attachment 173097 [details]
Updated svn diff file to update lang/qore

This patch obsoletes the first patch submitted.
Comment 7 estrabd 2016-07-29 15:59:29 UTC
I updated this diff to reflect the suggestions in Comment #3 below.

I tested this with both poudriere and portlint, they were in conflict with warnings so I deferred to what made portlint happy.

The new patch obsoletes the first patch submitted.
Comment 8 commit-hook freebsd_committer freebsd_triage 2016-08-25 03:00:42 UTC
A commit references this bug:

Author: wen
Date: Thu Aug 25 03:00:13 UTC 2016
New revision: 420832
URL: https://svnweb.freebsd.org/changeset/ports/420832

Log:
  - Update to 0.8.12
  - Switch to USE_GITHUB
  - Add missing BUILD_DEPENDS

  PR:		210447
  Submitted by:	estrabd@gmail.com(maintainer)

Changes:
  head/lang/qore/Makefile
  head/lang/qore/distinfo
  head/lang/qore/pkg-plist
Comment 9 Wen Heping freebsd_committer freebsd_triage 2016-08-25 03:03:47 UTC
Hi,

   In my last commit I install qore.pc into libdata/pkgconfig, which fix the problem mentioned by pawel@ in comment #1.
   And I suggest remove the line cd ${WRKSRC}; ./reconf.sh which is not needed, would you test it ?

wen