Skip to content

Fix Linux GDAL build and harden Windows publish#2048

Merged
MJohnson459 merged 1 commit into
OpenDroneMap:masterfrom
MJohnson459:investigate-release-3.6.1
Jul 15, 2026
Merged

Fix Linux GDAL build and harden Windows publish#2048
MJohnson459 merged 1 commit into
OpenDroneMap:masterfrom
MJohnson459:investigate-release-3.6.1

Conversation

@MJohnson459

@MJohnson459 MJohnson459 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

In preparation for a release, I am going through the different builds to try and make them reliable. This PR includes three related fixes that fix the Docker and WSL builds.

  • configure.sh: expose the SuperBuild gdal-config so Fiona and rasterio build against the compiled GDAL 3.11.1 instead of failing to find gdal-config on Linux/Docker. (e.g https://github.com/OpenDroneMap/ODM/actions/runs/28655904272)
  • publish-windows.yml: upload the installer artifact with if: always() and add a concurrency group so a tagged build is not superseded.
  • Add RELEASE.md documenting the release process and artifact checklist.

- configure.sh: expose the SuperBuild gdal-config so Fiona and rasterio
  build against the compiled GDAL 3.11.1 instead of failing to find
  gdal-config on Linux/Docker.
- publish-windows.yml: upload the installer artifact with if: always()
  and add a concurrency group so a tagged build is not superseded.
- Add RELEASE.md documenting the release process and artifact checklist.
Comment thread configure.sh
# Fiona and rasterio build from source and locate GDAL through gdal-config,
# which SuperBuild installs under SuperBuild/install/bin.
export GDAL_CONFIG="${RUNPATH}/SuperBuild/install/bin/gdal-config"
export PATH="${RUNPATH}/SuperBuild/install/bin:$PATH"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could swear I dealt with this one before; maybe it was in configure.py or the Docker?

@DodgySpaniard DodgySpaniard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MJohnson459
MJohnson459 merged commit b3af418 into OpenDroneMap:master Jul 15, 2026
2 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