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.

Sign up for GitHub

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

Jump to bottom

Use GDALGetMetadataItem() instead of GDALGetMetadata() on drivers #55115

Merged

nyalldawson merged 1 commit into qgis:master from rouault:use_GDALGetMetadataItem

Nov 2, 2023

Contributor

rouault commented Nov 1, 2023

There was nothing wrong per-se, but using GDALGetMetadataItem() will make QGIS compatible of the in-development GDAL capability of delaying the actual loading of GDAL plugin drivers.
Cf https://github.com/OSGeo/gdal/compare/master...rouault:gdal:deferred_plugin?expand=1 (GDAL RFC to follow soon)


          Use GDALGetMetadataItem() instead of GDALGetMetadata() on drivers

9a7c00e

There was nothing wrong per-se, but using GDALGetMetadataItem() will
make QGIS compatible of the in-development GDAL capability of delaying
the actual loading of GDAL plugin drivers.
Cf https://github.com/OSGeo/gdal/compare/master...rouault:gdal:deferred_plugin?expand=1
(GDAL RFC to follow soon)

github-actions bot added this to the 3.36.0 milestone

nyalldawson approved these changes

View reviewed changes

nyalldawson merged commit 1e4f7a2 into qgis:master

32 checks passed

Contributor Author

rouault commented Nov 2, 2023

Related GDAL RFC in OSGeo/gdal#8648

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet