Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make all OGR drivers optional (but the Memory one)! #10856

Merged
merged 13 commits into from
Sep 30, 2024

Conversation

rouault
Copy link
Member

@rouault rouault commented Sep 22, 2024

  • Make OGR_VRT driver optional and buildable as plugin
  • Make KML driver optional and buildable as plugin
  • Make Shapefile driver optional and buildable as plugin (http://switchfromshapefile.org/ you're happy, aren't you ?). Note that if you disable the shapefile driver, the autotest suite will fail on a significant number of tests...
  • Make GeoJSON (and companion TopoJSON, ESRIJSON, GeoJSONSeq) drivers optional or buildable as a plugin (if not builtin, the JSONFG driver is not available)
  • Make MapInfo driver optional or buildable as a plugin (if not builtin, Shapefile attribute indices are not available)

A build with -DGDAL_BUILD_OPTIONAL_DRIVERS=OFF -DOGR_BUILD_OPTIONAL_DRIVERS=OFF now leads to:

$ ogrinfo --formats
Supported Formats: (ro:read-only, rw:read-write, +:update, v:virtual-I/O s:subdatasets)
  Memory -vector- (rw+): Memory

@rouault rouault added this to the 3.10.0 milestone Sep 22, 2024
@rouault rouault force-pushed the more_optional_drivers branch 6 times, most recently from 46f5ff9 to 164906b Compare September 22, 2024 02:25
@rouault rouault force-pushed the more_optional_drivers branch 3 times, most recently from 331a206 to dd6bc7e Compare September 22, 2024 03:02
…ptional or buildable as a plugin (if not builtin, the JSONFG driver is not available)
…, Northwoord driver is not available, and Shapefile attribute indices too)
ogr/ogrsf_frmts/CMakeLists.txt Outdated Show resolved Hide resolved
ogr/ogrsf_frmts/CMakeLists.txt Outdated Show resolved Hide resolved
@rouault rouault merged commit 092b1b3 into OSGeo:master Sep 30, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants