Opened 13 years ago
Closed 13 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: |
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 (3)
comment:1 by , 13 years ago
comment:2 by , 13 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 , 13 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
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.
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...