docker: build gdal-grass with CMake#5106
Conversation
After gdal-grass' drop of Autotools support with OSGeo/gdal-grass@fa4b045
neteler
added
the
backport to 8.4
|
Thanks! To make G84 happy as well we'll need to backport this PR once merged. |
|
Fixes #5107 |
|
We need to have a representative the Docker build (e.g. the one from root dir Dockerfile) build for PRs for clarity. It may also be a good idea to not use latest version in the clone, but a release instead. |
wenzeslaus
left a comment
wenzeslaus
left a comment
There was a problem hiding this comment.
Well, the build is broken and the CI tests that only on main and we have already bunch of commits there failing with this error. Let's merge it.
These changes should work for latest release too, if we were to change version. |
|
Ahh, Alpine didn't install cmake. I'll fix that. |
|
@nilason shall we keep the hardcoded-version workaround for G84 or shall anything from main be backported? |
I don't know. I never tested if gdal-grass driver actually worked before without hardcoded paths. But backporting that part wouldn't hurt either. |
Partial, manual backport of: OSGeo#5106 1c1d01d OSGeo#5108 6b652b6 OSGeo#5115 d657e54 OSGeo#5137 7bfe274
Partial, manual backport of: OSGeo#5106 1c1d01d OSGeo#5108 6b652b6 OSGeo#5115 d657e54 OSGeo#5137 7bfe274
Partial, manual backport of: OSGeo#5106 1c1d01d OSGeo#5108 6b652b6 OSGeo#5115 d657e54 OSGeo#5137 7bfe274
nilason
removed
the
backport to 8.4
3 participants
After gdal-grass' drop of Autotools support with
OSGeo/gdal-grass@fa4b045.
I can't test these changes, but it should be good. Either someone could test it manually, or we merge and see...