Skip to content

add libffi as direct dependency in recent LLVM easyconfigs#23429

Merged
ocaisa merged 3 commits intoeasybuilders:developfrom
Crivella:fix-LLVM_ffi
Jul 25, 2025
Merged

add libffi as direct dependency in recent LLVM easyconfigs#23429
ocaisa merged 3 commits intoeasybuilders:developfrom
Crivella:fix-LLVM_ffi

Conversation

@Crivella
Copy link
Copy Markdown
Contributor

@Crivella Crivella commented Jul 16, 2025

Companion PR to

Notes

  • libffi was already an indirect dependency but this future proofs it and makes it clear it is also a dependency
  • Removed the skip_mod_files_sanity_check option as it is now being set by the EasyBlock when needed
  • Even if the linked issue is related to offload which has been introduced since LLVM >= 19 the added configure options are also available for 18.1.8

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#3849
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jsczen3c2.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/b9ec75d5de315a6974d83c5997c76978 for a full test report.

@Crivella
Copy link
Copy Markdown
Contributor Author

@boegelbot please test @ jsc-zen3
EB_ARGS="--installpath /tmp/$USER/pr-23429 --include-easyblocks-from-pr 3849"
CORE_CNT=16

@boegelbot
Copy link
Copy Markdown
Collaborator

@Crivella: 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=23429 EB_ARGS="--installpath /tmp/$USER/pr-23429 --include-easyblocks-from-pr 3849" EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_23429 --ntasks="16" ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 3078300397 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
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#3849
SUCCESS
Build succeeded for 5 out of 5 (5 easyconfigs in total)
jsczen3c2.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/5ad479af7da963a53922c11d91df5a63 for a full test report.

@bartoldeman
Copy link
Copy Markdown
Contributor

Test report for LLVM-20.1.5-GCCcore-13.3.0.eb
https://gist.github.com/bartoldeman/ad4001205949db39be05bf186e80f356

two notes:

  • I tweaked the easyblock a bit to not include the GCC lib dir into LD_LIBRARY_PATH, as that breaks in our somewhat unusual setup (using GCC from Gentoo prefix).
  • I raised the number of allowed test failures to 50 as a precaution and got:
    Test suite completed with 29 failed tests (50 allowed)

Copy link
Copy Markdown
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

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

LGTM

@ocaisa ocaisa merged commit 46bad6d into easybuilders:develop Jul 25, 2025
8 checks passed
@Crivella Crivella deleted the fix-LLVM_ffi branch July 25, 2025 13:41
@boegel boegel changed the title LLVM fix for libffi add libffi as direct dependency in recent LLVM easyconfigs Jul 31, 2025
@boegel boegel added this to the next release (5.1.2) milestone Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants