Skip to content

Updating superbuild GDAL configuration

Antoine Regimbeau requested to merge gdal_freexl into develop

Summary

On a environment containing libfreexl gdal automatically link with it. Whereas there is no need and package are delivered without it.

Rationale

There is no need for this. The configure command of GDAL has been updated to avoid this link. I'd like to take this opportunity to make sure that our GDAL is as light as possible. Here is the list of option that we can use in GDAL (there might be more...) :

GDAL configure option OTB Superbuild value
geos with
geotiff with
hide-internal-symbols none (default without)
libtiff with
libz with
threads with
bsb none
cfitsio without
cryptopp none (default with)
curl with
ecw without
expat with
fme without
freexl without (once this is merged)
gif without
gnm none (default with)
grass without
grib none (default without)
hdf4 with
hdf5 with
idb without
ingres without
jasper without
jp2mrsid without
jpeg with
kakadu without
libgrass without
libkml without
mrf none (default with)
mrsid without
mysql without
netcdf with
odbc without
ogdi without
openjpeg with
pcidsk with
pcraster without
pcre without
perl without
pg without
php without
png without
python without
qhull internal
sde without
sqlite3 none (default with)
webp without
xerces without
xml2 without
spatialite without

Those without value are a real problem because behavior depends on the plateform they are built on. GDAL configure script is testing whereas the system have the option or not and then use it (most of the time). That is why packaging on hulk are broken, GDAL is using freexl.
Anyway, as I was investigating I thought someone might have something to say about that, so here are the facts!

Here is a 🥔 for the long facts!

Copyright

The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement.


Check before merging:

  • All discussions are resolved
  • At least 2 👍 votes from core developers, no 👎 vote.
  • The feature branch is (reasonably) up-to-date with the base branch
  • Dashboard is green
  • Copyright owner has signed the ORFEO ToolBox Contributor License Agreement
Edited by Antoine Regimbeau

Merge request reports

Loading