Skip to content

Switch to using new spaces as delimiters for TCLLIBPATH#21724

Merged
boegel merged 4 commits intoeasybuilders:5.0.xfrom
Micket:tcllibpathdelimiter
Dec 21, 2024
Merged

Switch to using new spaces as delimiters for TCLLIBPATH#21724
boegel merged 4 commits intoeasybuilders:5.0.xfrom
Micket:tcllibpathdelimiter

Conversation

@Micket
Copy link
Copy Markdown
Contributor

@Micket Micket commented Oct 23, 2024

@Micket Micket added the bug fix label Oct 23, 2024
@Micket Micket added this to the 5.0 milestone Oct 23, 2024
@Micket Micket added the EasyBuild-5.0 EasyBuild 5.0 label Nov 13, 2024
@Micket
Copy link
Copy Markdown
Contributor Author

Micket commented Nov 13, 2024

@boegelbot please test @ jsc-zen3
EB_ARGS="--module-only BLT-20181223-GCCcore-11.2.0.eb bwidget-1.9.15-GCCcore-11.3.0.eb Graphviz-8.1.0-GCCcore-12.3.0.eb Tk-8.6.14-GCCcore-13.3.0.eb"
EB_BRANCH=5.0.x

@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 [[ "5.0.x" != 'develop' ]]; then EB_BRANCH="5.0.x" ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/"5.0.x" source init_env_easybuild_develop.sh; fi; EB_PR=21724 EB_ARGS="--module-only BLT-20181223-GCCcore-11.2.0.eb bwidget-1.9.15-GCCcore-11.3.0.eb Graphviz-8.1.0-GCCcore-12.3.0.eb Tk-8.6.14-GCCcore-13.3.0.eb" EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH="5.0.x" /opt/software/slurm/bin/sbatch --job-name test_PR_21724 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 2474136205 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
FAILED
Build succeeded for 0 out of 4 (4 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.4, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.18
See https://gist.github.com/boegelbot/b71fe3bcb6e5cf29edc240d9d50931be for a full test report.

@Micket
Copy link
Copy Markdown
Contributor Author

Micket commented Nov 13, 2024

@boegelbot please test @ jsc-zen3
EB_ARGS="BLT-20181223-GCCcore-11.2.0.eb bwidget-1.9.15-GCCcore-11.3.0.eb Graphviz-8.1.0-GCCcore-12.3.0.eb Tk-8.6.14-GCCcore-13.3.0.eb"
EB_BRANCH=5.0.x

@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 [[ "5.0.x" != 'develop' ]]; then EB_BRANCH="5.0.x" ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/"5.0.x" source init_env_easybuild_develop.sh; fi; EB_PR=21724 EB_ARGS="BLT-20181223-GCCcore-11.2.0.eb bwidget-1.9.15-GCCcore-11.3.0.eb Graphviz-8.1.0-GCCcore-12.3.0.eb Tk-8.6.14-GCCcore-13.3.0.eb" EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH="5.0.x" /opt/software/slurm/bin/sbatch --job-name test_PR_21724 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 2474238595 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
FAILED
Build succeeded for 0 out of 4 (4 easyconfigs in total)
jsczen3c2.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.4, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.18
See https://gist.github.com/boegelbot/cecefa5e0541e6914e216762ee99a0cc for a full test report.

@Micket
Copy link
Copy Markdown
Contributor Author

Micket commented Nov 13, 2024

Test report by @Micket
FAILED
Build succeeded for 6 out of 7 (4 easyconfigs in total)
vera-skylake-build - Linux Rocky Linux 8.9, x86_64, Intel Xeon Processor (Skylake, IBRS, no TSX), Python 3.6.8
See https://gist.github.com/Micket/a14372466007a762f2266788714534c5 for a full test report.

@Micket
Copy link
Copy Markdown
Contributor Author

Micket commented Nov 13, 2024

BLT source download failing, and I don't see how to obtain it.

Copy link
Copy Markdown
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

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

LGTM

@lexming
Copy link
Copy Markdown
Contributor

lexming commented Dec 19, 2024

@Micket you can go to https://sourceforge.net/p/blt/src/ci/b8cdc6c2cd8edba06ad53a5d56e961d35b07af28/tree/ and click on "Download snapshot"

@branfosj
Copy link
Copy Markdown
Member

@Micket you can go to https://sourceforge.net/p/blt/src/ci/b8cdc6c2cd8edba06ad53a5d56e961d35b07af28/tree/ and click on "Download snapshot"

I think sourceforge has caching on the generation of the zip file for download. If I try

wget https://sourceforge.net/code-snapshots/git/b/bl/blt/src.git/blt-src-b8cdc6c2cd8edba06ad53a5d56e961d35b07af28.zip

then that currently works. But I tried that 15 minutes ago and it failed.

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 20, 2024

There's a broader problem with BLT, it fails in the RPATH sanity check:

== FAILED: Installation ended unsuccessfully: Sanity check failed: Library Tcl30.so not found for
/software/BLT/20181223-GCCcore-11.2.0/lib/libBltTk30.so

Indeed:

ldd /software/BLT/20181223-GCCcore-11.2.0/lib64/libBltTk30.so | grep 'not found'
        Tcl30.so => not found

That has nothing to do with the changes in this PR though, so it's not a blocker

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 20, 2024

Test report by @boegel
SUCCESS
Build succeeded for 18 out of 18 (18 easyconfigs in total)
node3509.doduo.os - Linux RHEL 8.8, x86_64, AMD EPYC 7552 48-Core Processor, Python 3.6.8
See https://gist.github.com/boegel/ae27c178f8e8ae69e44700ada9adc4b3 for a full test report.

@boegel boegel merged commit 1443499 into easybuilders:5.0.x Dec 21, 2024
@boegel boegel removed the change label Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

5 participants