ci: repoint install-spatial-deps to @main - #33
Conversation
Every published tag -- v0.1 through v0.5, including the newest -- still runs `add-apt-repository -y ppa:ubuntugis/ubuntugis-unstable`. That installs libgdal37, while Posit's noble binary cache builds sf/terra against the Ubuntu-default libgdal34. The mismatch surfaces only at lazy-load, as `libgdal.so.34: cannot open shared object file`, and fails the whole job. Only @main dropped the PPA. Tags do not move, so pinning to one -- even the newest -- keeps the hazard. Was: v0.2 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
@achubaty I am not finished the full migration. Please do not work on this yet. |
|
Redundant with #31, which migrates this repo's The migration removes the Closing this one rather than #31 only because the migration is the superset; the repointing intent is identical and correct. Same treatment was applied to an equivalent set of repin PRs I had opened earlier. Not a comment on the ~40 module repos ( 🤖 Generated with Claude Code |
Repoints
install-spatial-depsfromv0.2to@main— 1 reference(s) in.github/workflows/R-CMD-check.yaml.Why
Every published tag of that action — v0.1 through v0.5, including the newest — still runs:
That PPA installs libgdal37 (GDAL 3.11.x). Posit's noble binary cache builds sf/terra against the Ubuntu-default libgdal34 — confirmed by
objdumpon the PPM noble tarballs, which needlibgdal.so.34,libproj.so.25,libgeos_c.so.1. The mismatch surfaces only at lazy-load, aslibgdal.so.34: cannot open shared object file, and takes the whole job with it.Only
@maindropped the PPA (PredictiveEcology/actions#25). Tags do not move, so pinning to one keeps the hazard — which is why "pin to the newest tag" is currently the wrong advice for this action.This repo's own history records the failure in both directions: a cached terra built against one libgdal while the runner supplied the other. The underlying cause was a stale
setup-r-dependenciespackage cache outliving a change in the runner's system GDAL — the PPA is what made the two disagree.Note
On the pak path this step is largely redundant anyway:
setup-r-dependencies→pak::lockfile_install()→install_sysreqs()already installslibgdal-dev,gdal-bin,libgeos-dev,libproj-dev,libsqlite3-devandlibudunits2-devfrom stock noble apt, and pak's own rules add the ubuntugis PPA only for Ubuntu 14.04/16.04. Removing the step entirely is a separate change, gated on a self-test run — this PR only makes the pin correct.🤖 Generated with Claude Code