Core depends on drivers (KML/GeoJson)

[wbaer]

Hi,

I wanted to testbuild 1.5b2 on wince and encountered unresolved symbol linker errors due to
a new dependency of the core ogrgeometry.cpp class on KML / GeoJson drivers for the exportToKML/Json methods.

Personally I would like to see export/import/conversion methods not in the ogrgeometry class
but in an own class with an own header file. But thats maybe something for an 1.6 release. Until then the methods in the ogrgeometry should be guarded by the appropriate ifdefs.

[wbaer]

ifdefs KML/GeoJSON export methods

[warmerdam]

A related error occurs if you build --without-ogr on linux.

/wrk/home/warmerda/wrk/gdal-1.5.0b2/.libs/libgdal.so: undefined reference to `AVCBinReadOpen'
/wrk/home/warmerda/wrk/gdal-1.5.0b2/.libs/libgdal.so: undefined reference to `AVCBinReadClose'
/wrk/home/warmerda/wrk/gdal-1.5.0b2/.libs/libgdal.so: undefined reference to `OGR_G_ExportToJson'
/wrk/home/warmerda/wrk/gdal-1.5.0b2/.libs/libgdal.so: undefined reference to `OGR_G_ExportToKML'
/wrk/home/warmerda/wrk/gdal-1.5.0b2/.libs/libgdal.so: undefined reference to `AVCBinReadNextTableRec'

[warmerdam]

I've skimmed the proposed patch, but I don't understand where the GEOJSON_ENABLED and KML_ENABLED macros are supposed to come from. In any event, the method must not be removed from the OGRGeometry API even if they are not implemented ... they should just issue a CPLError().

I have made some changes in ogrgeometry.cpp and aigdataset.cpp that fix the case where no OGR drivers are built and OGR_ENABLED is FALSE (r13334 in trunk), but this does not fix the problem on WinCE where it is common to build without most drivers. I'm turning this into a WinCE ticket, and turning over to mloskot to address as he sees fit (as long as it doesn't impact other platforms).

[warmerdam]

The WinCE port is currently orphaned for lack of developers.

[Jukka Rahkonen]

Unrealistic to believe that bugs in Windows CE port will be fixed.