Skip to content

drop GSL from GStreamer and move them to GCCcore toolchain#24179

Merged
branfosj merged 5 commits intoeasybuilders:developfrom
Micket:20251009155810_new_pr_GST-plugins-bad1266
Oct 12, 2025
Merged

drop GSL from GStreamer and move them to GCCcore toolchain#24179
branfosj merged 5 commits intoeasybuilders:developfrom
Micket:20251009155810_new_pr_GST-plugins-bad1266

Conversation

@Micket
Copy link
Copy Markdown
Contributor

@Micket Micket commented Oct 9, 2025

(created using eb --new-pr)

It's really frustrating that GStreamer being at GCC level pushes so several other things into GCC needlessly. GST-plugins, GTK4 etc.

These really do belong at core level, as they are in no way performance critical, and I trust GCC to be a much more reliable compiler for all the desktop GUI toolkit stuff.

GStreamer was only at GCC level due to GSL, but i traced down what it actually uses GSL for, and it's literally just some tests (that we don't even run!). Nothing in the install links to libgsl at all. I looked through the entire build log, so I'm pretty sure.
Nothing in GST-plugins either.

So, I propose we change this, move them down. Other things could be moved down if we do this.

I kept the GStreamer at version 1.26.5 for GCCcore 14.3.0, because that's what already in use, so that if someone has it build (I think we shipped it), it's going to stay compatible. So, I think this is safe to change even if someone doesn't rebuild their local GStreamer; they would just have one GCC and one GCCcore copy, and it doesn't matter which gets loaded)

@Micket Micket added the update label Oct 9, 2025
@Micket Micket added the change label Oct 9, 2025
@github-actions github-actions Bot removed the update label Oct 9, 2025
@Micket
Copy link
Copy Markdown
Contributor Author

Micket commented Oct 9, 2025

Test report by @Micket
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
vera-icelake-build - Linux Rocky Linux 9.6, x86_64, Intel(R) Xeon(R) Silver 4316 CPU @ 2.30GHz, Python 3.9.21
See https://gist.github.com/Micket/ee4f2d86fe6605e1b799f8e949541f4d for a full test report.

@Micket
Copy link
Copy Markdown
Contributor Author

Micket commented Oct 9, 2025

Test report by @Micket
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
vera-icelake-build - Linux Rocky Linux 9.6, x86_64, Intel(R) Xeon(R) Silver 4316 CPU @ 2.30GHz, Python 3.9.21
See https://gist.github.com/Micket/2c9f43939ee0fb9fcb05ad3fdc53f8ee for a full test report.

@Micket Micket added this to the next release (5.2.0?) milestone Oct 9, 2025
@Micket Micket added 2025a issues & PRs related to 2025a common toolchains 2025b issues & PRs related to 2025b common toolchains labels Oct 9, 2025
@Micket
Copy link
Copy Markdown
Contributor Author

Micket commented Oct 9, 2025

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

@Micket: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=24179 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_24179 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8251

Test results coming soon (I hope)...

Details

- notification for comment with ID 3387052452 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.6, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/386feeb4e7e6afdb13536257273be647 for a full test report.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 9, 2025

Test report by @Thyre
SUCCESS
Build succeeded for 12 out of 12 (6 easyconfigs in total)
Linux - Linux Arch Linux UNKNOWN, x86_64, AMD Ryzen 7 7800X3D 8-Core Processor (zen4), 1 x AMD Navi 48 [Radeon RX 9070/9070 XT/9070 GRE] (device id: 0x7550, gfx: gfx1201, driver: 6.17.1-arch1-1), 1 x AMD Raphael (device id: 0x164e, gfx: gfx1036, driver: 6.17.1-arch1-1), Python 3.13.7
See https://gist.github.com/Thyre/2b71ed581ed75a091a246c753b2fb0b1 for a full test report.

@Thyre
Copy link
Copy Markdown
Collaborator

Thyre commented Oct 9, 2025

Doesn't seem to cause issues with a system GSL installed (as on my machine). I generally prefer this, as this makes handling quite a few things easier...

@branfosj
Copy link
Copy Markdown
Member

Test report by @branfosj
SUCCESS
Build succeeded for 11 out of 11 (6 easyconfigs in total)
bear-pg0105u03a - Linux RHEL 8.10, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/branfosj/888a51c8655a3bba6044b4fb32e46a4d for a full test report.

@branfosj
Copy link
Copy Markdown
Member

Test report by @branfosj
SUCCESS
Build succeeded for 15 out of 15 (6 easyconfigs in total)
bear-pg0210u03a - Linux RHEL 8.10, x86_64, Intel(R) Xeon(R) Platinum 8480CL (sapphirerapids), Python 3.6.8
See https://gist.github.com/branfosj/52c1b9ef58fc04c0deaf73361a33c318 for a full test report.

@branfosj
Copy link
Copy Markdown
Member

Going in, thanks @Micket!

@branfosj branfosj merged commit 530a683 into easybuilders:develop Oct 12, 2025
8 checks passed
@Micket Micket deleted the 20251009155810_new_pr_GST-plugins-bad1266 branch October 12, 2025 12:44
@boegel boegel changed the title Drop GSL from GStreamer and move them to GCCcore drop GSL from GStreamer and move them to GCCcore toolchain Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2025a issues & PRs related to 2025a common toolchains 2025b issues & PRs related to 2025b common toolchains change ready-to-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants