grass icon indicating copy to clipboard operation
grass copied to clipboard

[Bug] Docker builds fail to compile PDAL 2.7.2 from source, missing release assets

Open echoix opened this issue 1 year ago • 4 comments

Describe the bug

PDAL released 2.7.2 today.

Contrary to the other releases, that new release doesn't include the same assets that are used in build scripts. Thus, downloading with an url similar to https://github.com/PDAL/PDAL/releases/download/${PDAL_VERSION}/PDAL-${PDAL_VERSION}-src.tar.gz will fail.

To reproduce

See a failing Docker build here https://github.com/OSGeo/grass/actions/runs/9719096265/job/26828288072#step:7:3154, from this PR https://github.com/OSGeo/grass/pull/3937

Expected behavior

Have PDAL build correctly from source, either with another URL, or have the docker builds be ran before being merged.

Screenshots

System description

Additional context

Filed an issue in the PDAL repo here: https://github.com/PDAL/PDAL/issues/4432

echoix avatar Jun 28 '24 22:06 echoix

Yes it was me that merged the failing PR that could be reverted if you think its too impactful (for now its red statuses and no new uploads for 2 of the 4 docker images). But our workflows should be able to handle this, without having to do much work in parallel for some PRs. So I just went with it, as its currently the "designed" way to handle changes in dockerfiles.

echoix avatar Jun 28 '24 22:06 echoix

Seems to be fixed upstream in PDAL, a few hours ago.

neteler avatar Jun 29 '24 05:06 neteler

Now, only the ubuntu_wxgui build fails, as the image has PROJ 8.2.1 (#8 82.88 Setting up libproj-dev:amd64 (8.2.1-1) ...), while debian has 9.1.1 #8 48.26 Setting up libproj-dev:amd64 (9.1.1-1+b1) ...)

I know that the cmake file to find proj changed in 2.7.2, but they also relaxed from requiring PROJ 9.2+ to only PROJ 9.0+

echoix avatar Jun 29 '24 18:06 echoix

See https://github.com/OSGeo/grass/pull/3946

echoix avatar Jun 30 '24 12:06 echoix