Ticket #1660 (closed defect: fixed)

Opened 15 months ago

Last modified 15 months ago

Be louder about GeoJSON support

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS 2.0.0
Component: build/upgrade/install Version: trunk
Keywords: Cc:

Description

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

Changed 15 months ago by strk

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...

Changed 15 months ago by strk

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

Changed 15 months ago by strk

  • status changed from new to closed
  • resolution set to fixed

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.