SyneRBI / SIRF

Main repository for the CCP SynerBI software
http://www.ccpsynerbi.ac.uk
Other
59 stars 29 forks source link

[GHA] test with gcc 8 and 9 #1142

Closed KrisThielemans closed 1 year ago

KrisThielemans commented 1 year ago

port some changes from the SIRF-SuperBuild.

Fixes #1140, as we no longer assume gcc-8 is installed.

KrisThielemans commented 1 year ago

fails with

Error: .github/workflows/build-test.yml (Line: 94, Col: 30): A sequence was not expected
Error: .github/workflows/build-test.yml (Line: 95, Col: [2](https://github.com/SyneRBI/SIRF/actions/runs/3291458693/jobs/5425633867#step:6:2)9): A sequence was not expected

but it seems to be valid YAML. @paskino any idea?

KrisThielemans commented 1 year ago

https://rhysd.github.io/actionlint/ gave some (cryptic) errors on the use of ${{ matrix.blabla }} so I've got rid of then env entirely

KrisThielemans commented 1 year ago

The Gadgetron version that we use fails to compile with gcc 10, so disabling that