Opened 12 years ago

Closed 12 years ago

#1660 closed defect (fixed)

Be louder about GeoJSON support

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS 2.0.0
Component: build Version: master
Keywords: Cc:


I noticed I wasn't using GeoJSON only today. ./configure isn't loud enough about it, should we raise a WARNING ? Should it be the default like topology and raster and fail on missing requirement ?

Also, README.postgis, in the REQUIREMENTS section, doesn't mention libjson-c _at_all_

Change History (3)

comment:1 by strk, 12 years ago

r9416 updates the REQUIREMENTS section of README.postgis. I'll note this is another case of redundancy, the Requirements section in the manual was more up-to-date.

I'd split install instructions out of README.postgis, btw, and into an INSTALL file. You don't need to install the INSTALL file…

comment:2 by strk, 12 years ago

With r9417 I've added apt-based command to install required library. I saw that postgis_full_version() isn't reporting libjson-c version

comment:3 by strk, 12 years ago

Resolution: fixed
Status: newclosed

Another signature (sorry) added in r9418: postgis_libjson_version() The new function is then used by postgis_full_version() to report libjson availability (not really version as the library doesnt' expose one!!).

NOTE: the new function is not documented (and maybe shouldn't be).

I'll consider this closed for the time being.

Note: See TracTickets for help on using tickets.