Opened 19 years ago
Last modified 16 years ago
#822 closed enhancement (fixed)
GRASS5.7+ coordinate system patch
Reported by: | warmerdam | Owned by: | warmerdam |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | GDAL_Raster | Version: | unspecified |
Severity: | minor | Keywords: | grass |
Cc: | Markus Neteler, Mateusz Łoskot |
Description (last modified by )
Frank, find attached the patch to enable projection support in the GRASS/GDAL plugin. It's fairly straightforward: - added libgrass_gproj.so to the build process - copies during compilation the files datum.table + ellipse.table from GRASS source code to /usr/.../share/ directory, within that I generate grass/etc/ to keep the structure clean. These files are used by GPJ_grass_to_wkt() That's it. I don't face problems of circular dependencies at it was once for Radim (maybe things changed in GRASS, I don't remember). TODOs: * gdal/frmts/grass/pkg/configure: please run autoconf as I have an older version * gdal/frmts/grass/grass57dataset.cpp: - hardcoded path to fake_gisbase - maybe hasGisbase can be retired, I left it for now Hopefully it works as well for you so that GDAL (so also QGIS, Mapserver etc) see the GRASS map projection. Cheers Markus
Attachments (2)
Change History (7)
by , 19 years ago
Attachment: | grass_driver.diff added |
---|
comment:1 by , 19 years ago
Frank, would you mind to apply this patch? Would solve - problem for gdal - problem for the new R-stats interface - problem for QGIS I am still happy with it... Thanks Markus
comment:2 by , 19 years ago
Frank, let me express my interest to get this patch applied :-) Thanks Markus
comment:3 by , 19 years ago
Markus, I have tried it in my regular GDAL build, and it causes a configure failure when testing for Xerces. This appears to be due to grass_gproj linking back to libgdal.so. checking for Xerces C++... requested, but compile or link fails! g++ -I/usr/local/include -I/usr/local/include/xercesc -D_REENTRANT -o conftest conftest.cpp -L/usr/local/lib -lxerces-c -lpthread -L/u/pkg/Geo_DSDK-4.0.10.720/lib/Release -lltidsdk -lpthread -L/u/pkg/HDF42_inst/lib -lmfhdf -ldf -logdi31 -lcsf -L/usr/local/grass-6.0.0/lib -lgrass_I -lgrass_vask -lgrass_gmath -lgrass_gis -lgrass_datetime -lgrass_gproj -lz -lpthread -lm -lrt -ldl -L/u/pkg/kakadu-4.1.0/lib -lkdu /home/warmerda/gdal/libgdal.so: warning: the 're_max_failures' variable is obsolete and will go away. configure: error: Please correct Xerces options, or don't enable Xerces. I imagine this isn't a problem for building GRASS support as a plugin, but it does - at least under some circumstances - screw it up for direct use in GDAL. It might make sense to have the build logic "know" whether to use the grass_gproj functions based on whether it is built in or not. I'm not sure to what extent this will be an issue on other platforms. For now I am backing the patch out again.
comment:4 by , 19 years ago
Patch incorporated as is. It will be in GDAL 1.3.1. I have also rebuilt the plugin package and included the OGR GRASS driver. http://www.gdal.org/dl/gdal-grass-1.3.1.tar.gz
comment:5 by , 16 years ago
Cc: | added |
---|---|
Description: | modified (diff) |
Keywords: | grass added |
Note:
See TracTickets
for help on using tickets.
patch