Bug 211203
| Summary: | databases/postgis22: compile error: unknown type name 'LWORD' | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Robert Kruus <robert.kruus> | ||||||
| Component: | Individual Port(s) | Assignee: | freebsd-ports-bugs (Nobody) <ports-bugs> | ||||||
| Status: | Closed FIXED | ||||||||
| Severity: | Affects Only Me | CC: | cmt, rhurlin, vlad-fbsd | ||||||
| Priority: | --- | Keywords: | patch | ||||||
| Version: | Latest | Flags: | vlad-fbsd:
maintainer-feedback+
koobs: merge-quarterly? |
||||||
| Hardware: | Any | ||||||||
| OS: | Any | ||||||||
| Bug Depends on: | 214906 | ||||||||
| Bug Blocks: | |||||||||
| Attachments: |
|
||||||||
|
Description
Robert Kruus
2016-07-18 16:18:57 UTC
Thanks for your analysis and suggestion. I was just preparing exactly the same patch, you published ;) Some short test of the patched databases/postgis22 did no show any misbehaviour to me. So probably, the patch should be ok? If you could confirm, that the patched version *works* for you, I would set the maintainer-feedback flag to '+', thanks. Seems to work fine here -- though I am just really using it for point data so no real extensive testing done. Comment on attachment 172667 [details]
Patch for force include the headers from build directory first.
@Rainer, you should be able to set maintainer-approval to + on the attachment now, once QA is confirmed
A unified diff against the port (via make makepatch) would also be good.
Created attachment 172725 [details]
patch to circumstance the update dilemma
Ok, thanks Robert for trying and Kubilay for his incitations.
I did some tests from within graphics/qgis and math/saga and it seems to work there, too.
QA:
- A new unified diff is attached, tested it with Poudriere (9.3i+a, 10.3i+a and HEADi+a).
- 'portlint -AC' gives some warnings about relativ paths, but this is in the replacing section.
So it should be ready for commit now.
@Rainer great stuff :) this has been fixed upstream https://trac.osgeo.org/postgis/ticket/3604 and made it's way into 2.2.3 - so this bug will be obsoleted by bug #214906 (In reply to Christoph Moench-Tegeder from comment #6) As the maintainer of databases/postgis22, I also would suggest to close this bug and to commit the patch from bug #214906 instead. Many thanks in advance. A commit references this bug: Author: cmt Date: Wed Nov 30 21:51:08 UTC 2016 New revision: 427461 URL: https://svnweb.freebsd.org/changeset/ports/427461 Log: update databases/postgis22 to 2.2.4 also, fixes build when older version is installed[1] PR: 214906, 211203 [1] Reported by: robert.kruus@utoronto.ca [1] Approved by: rhurlin@gwdg.de (maintainer), rene (mentor) MFH: 2016Q4 Changes: head/databases/postgis22/Makefile head/databases/postgis22/distinfo head/databases/postgis22/pkg-plist fixed by ports r427461 A commit references this bug: Author: cmt Date: Thu Dec 1 10:19:51 UTC 2016 New revision: 427484 URL: https://svnweb.freebsd.org/changeset/ports/427484 Log: MFH: r427461 update databases/postgis22 to 2.2.4 also, fixes build when older version is installed[1] PR: 214906, 211203 [1] Reported by: robert.kruus@utoronto.ca [1] Approved by: rhurlin@gwdg.de (maintainer), rene (mentor) Approved by: ports-secteam (junovitch) Changes: _U branches/2016Q4/ branches/2016Q4/databases/postgis22/Makefile branches/2016Q4/databases/postgis22/distinfo branches/2016Q4/databases/postgis22/pkg-plist |
