Opened 7 years ago

Closed 7 years ago

#55244 closed defect (fixed)

postgis2 2.3.2: make postgis function "ST_GeomFromGeoJSON" work by adding the json-c port as dependency

Reported by: mjl- Owned by: Veence (Vincent)
Priority: Normal Milestone:
Component: ports Version: 2.4.2
Keywords: haspatch Cc:
Port: postgis2

Description

Currently, the postgis function ST_GeomFromGeoJSON fails with an error. See below:

testdb=> select ST_GeomFromGeoJSON('{"type": "Point", "coordinates": [-105.01621, 39.57422]}');
ERROR:  You need JSON-C for ST_GeomFromGeoJSON

To reproduce the above error, run the query in any database that has the postgis extension installed ("create extension postgis;").

After applying the attached diff and running ST_GeomFromGeoJSON again:

testdb=> select ST_GeomFromGeoJSON('{"type": "Point", "coordinates": [-105.01621, 39.57422]}');
             st_geomfromgeojson
--------------------------------------------
 010100000090F7AA9509415AC0C45A7C0A80C94340
(1 row)

Attached is an updated Portfile and a diff taking you from current/old Portfile to the attached Portfile.

From what I can gather from earlier trac tickets, postgis+json-c builds were broken in the past. However, that was a long time ago and no longer the case:

https://lists.macports.org/pipermail/macports-users/2014-August/036085.html https://trac.macports.org/ticket/44124

I only tested this version/variant: postgis2 @2.3.2_0+postgresql96+raster+topology

Attachments (2)

Portfile (4.6 KB) - added by mjl- 7 years ago.
updated Portfile
Portfile.diff (723 bytes) - added by mjl- 7 years ago.
diff from old/current portfile to updated portfile

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by mjl-

Attachment: Portfile added

updated Portfile

Changed 7 years ago by mjl-

Attachment: Portfile.diff added

diff from old/current portfile to updated portfile

comment:1 Changed 7 years ago by Veence (Vincent)

Thanks. Matter of fact, I'm going to upgrade to PostGIS 2.4 and take this into account.

comment:2 Changed 7 years ago by Veence (Vincent)

Should be done. Please test.

comment:3 Changed 7 years ago by mf2k (Frank Schima)

Cc: Veence removed
Owner: set to Veence
Status: newassigned
Type: enhancementdefect

comment:4 Changed 7 years ago by Veence (Vincent)

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.