[GDAL] Remove giflib to fix Windows

[visr]

Fixes #8781. I rebuilt with different combinations of the 7 dependencies added in #8749 to find that giflib breaks GDAL on Windows. So I suggest removing this dependency for now. @eschnett are you ok with this? I should add that installing the older Giflib_jll v5.2.1 (not the newly built v5.2.2) with the current GDAL_jll also didn't help.

[giordano]

I'm confused by your message

I should add that installing the older Giflib_jll v5.2.1 (not the newly built v5.2.2) with the current GDAL_jll also didn't help.

and the comment in the code

# Breaks GDAL on Windows as of Giflib_jll v5.2.2 (#8781)

as I find the two statements somewhat contradicting.

[visr]

Yeah I assume that GDAL was built against v5.2.2 (with a 5.2.1 compat). I can load both giflib JLLs fine. And if I have either version in my environment at load time, it both fails to load the current GDAL_jll.

Or does it build against the oldest compatible dependencies, in which case we should try setting compat to 5.2.2.

With this PR build the GDAL.jl tests pass on Windows except for two minor GDAL 3.9 changes.

[evetion]

If you look at the GMT failure, it already failed with Giflib_jll v5.2.1+0.

Another interesting thing is that if you put all required libraries in the same location (via JLLPrefixes.deploy_artifact_paths), you can load libgdal.dll just fine. Might be a red herring, but it seems to be some relative load thing at init time. Are there any tools to inspect/debug the dll dep tree like from within Julia?

@visr Did you also check with DGDAL_USE_WEBP=OFF (while including giflib), as libwebp also depends on giflib?

[giordano]

Are there any tools to inspect/debug the dll dep tree like from within Julia?

https://github.com/giordano/DependencyWalker.jl?

[evetion]

Are there any tools to inspect/debug the dll dep tree like from within Julia?

giordano/DependencyWalker.jl?

Excellent stuff, this will save us some time in the future! That does indeed simply say libgif.dll not found.

[eschnett]

This looks good. I can't tell whether there are any users of GDAL relying to giflib, though.

[giordano]

Yeah I assume that GDAL was built against v5.2.2 (with a 5.2.1 compat)

If there's only a compat bound, then the minimum version matching it is used for building: https://docs.binarybuilder.org/stable/reference/#BinaryBuilderBase.Dependency

[visr]

Ok then I'd say let's merge to fix the breakage, we can look into ways to get giflib working later.