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.

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

GDAL 3.5.0 #136

Merged
merged 2 commits into from
Jun 2, 2022
Merged

GDAL 3.5.0 #136

merged 2 commits into from
Jun 2, 2022

Conversation

visr
Copy link
Member

@visr visr commented Jun 2, 2022

visr added 2 commits June 2, 2022 16:02
@visr
set compat to GDAL_jll 3.5
1397f0b 
This in turn requires PROJ_jll 900, so may as well specify that here as well.

There are some removed drivers as indicated in https://github.com/OSGeo/gdal/blob/v3.5.0/NEWS.md. I guess if GDAL doesn't consider that breaking, neither will we.

Bumped a minor version for new features.
@visr
wrap GDAL 3.5.0
3fc5252
@visr
Copy link
Member Author

visr commented Jun 2, 2022 •
edited
Loading

Unfortunately this build seems broken for macOS on Julia 1.6 and 1.7:

ERROR: LoadError: InitError: could not load library "/Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib"
dlopen(/Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib, 1): Library not loaded: @rpath/libcurl.4.dylib
  Referenced from: /Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib
  Reason: Incompatible library version: libgdal.31.dylib requires version 13.0.0 or later, but libcurl.4.dylib provides version 12.0.0

Luckily because of the PROJ_jll compat settings in this package people won't generally get this build.

visr added a commit to visr/Yggdrasil that referenced this pull request Jun 2, 2022
@visr
[GDAL] add LibCURL_jll compat
6fce471 
Should fix

```
ERROR: LoadError: InitError: could not load library "/Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib"
dlopen(/Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib, 1): Library not loaded: @rpath/libcurl.4.dylib
  Referenced from: /Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib
  Reason: Incompatible library version: libgdal.31.dylib requires version 13.0.0 or later, but libcurl.4.dylib provides version 12.0.0
```

from JuliaGeo/GDAL.jl#136

Version 7.73 is the version that julia 1.6 uses:
https://github.com/JuliaLang/julia/blob/v1.6.6/stdlib/LibCURL_jll/Project.toml#L3
@visr visr mentioned this pull request Jun 2, 2022
Merged
giordano pushed a commit to JuliaPackaging/Yggdrasil that referenced this pull request Jun 2, 2022
@visr
[GDAL] add LibCURL_jll compat (#4981)
088cbff 
Should fix

```
ERROR: LoadError: InitError: could not load library "/Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib"
dlopen(/Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib, 1): Library not loaded: @rpath/libcurl.4.dylib
  Referenced from: /Users/runner/.julia/artifacts/83ad874b4fee57727baf6ef9ca825d716af58f20/lib/libgdal.31.dylib
  Reason: Incompatible library version: libgdal.31.dylib requires version 13.0.0 or later, but libcurl.4.dylib provides version 12.0.0
```

from JuliaGeo/GDAL.jl#136

Version 7.73 is the version that julia 1.6 uses:
https://github.com/JuliaLang/julia/blob/v1.6.6/stdlib/LibCURL_jll/Project.toml#L3
@visr
Copy link
Member Author

visr commented Jun 2, 2022

Fixed after rebuilding GDAL with a higher libcurl compat.

@visr visr merged commit 68d8c3f into master Jun 2, 2022
@visr visr deleted the gdal35 branch June 2, 2022 17:41
@visr visr mentioned this pull request Jun 2, 2022
Closed
@evetion
Copy link
Member

evetion commented Jun 2, 2022

Nice, this should come with geoparquet support!

@visr
Copy link
Member Author

visr commented Jun 2, 2022 •
edited
Loading

From the release notes:

Add read/write OGR Parquet (Apache Parquet) and 'Arrow' (Apache Arrow IPC File/Feather or stream) drivers. Only in CMake builds

I didn't switch the build over to CMake yet, so they are not available. Also haven't checked what dependencies are needed for them to build.

@joa-quim
Copy link
Contributor

joa-quim commented Jun 4, 2022

I built GDAL with cmake a couple weeks ago and for geoparquet there is the extra dependency of the arrow lib (it's documented in GDAL) which looks like a complicated beast with many possible building options. I just used the basic ones and everything built, but did not do any further tests.

@evetion
Copy link
Member

evetion commented Jun 4, 2022

So the cmake build would also need to depend on something like JuliaPackaging/Yggdrasil#918

@visr visr mentioned this pull request Aug 19, 2022
Open
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@visr @evetion @joa-quim