Skip to content

add patch specified via patch_ctypes_ld_library_path to patches in constructor of Python easyblock#3965

Merged
casparvl merged 1 commit intoeasybuilders:developfrom
boegel:20251022155908_new_pr_python
Oct 22, 2025
Merged

add patch specified via patch_ctypes_ld_library_path to patches in constructor of Python easyblock#3965
casparvl merged 1 commit intoeasybuilders:developfrom
boegel:20251022155908_new_pr_python

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Oct 22, 2025

(created using eb --new-pr)

follow-up for:

Doing the injection in the constructor makes sense, and it avoid problems with check_sha256_checksums reporting that there's one too many checksums specified in the easyconfig file (cfr. easybuilders/easybuild-easyconfigs#23499)

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Oct 22, 2025

@boegelbot please test @ jsc-zen3
CORE_CNT=16
EB_ARGS="Python-3.12.3-GCCcore-13.3.0.eb --installpath /tmp/$USER/pr3965"

@boegel boegel changed the title add patch specified via patch_ctypes_ld_library_path to patches in constructor of Python easyblock add patch specified via patch_ctypes_ld_library_path to patches in constructor of Python easyblock Oct 22, 2025
@boegelbot
Copy link
Copy Markdown

@boegel: 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=3965 EB_ARGS="Python-3.12.3-GCCcore-13.3.0.eb --installpath /tmp/$USER/pr3965" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3965 --ntasks="16" ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 3432527470 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

Test report by @boegelbot

Overview of tested easyconfigs (in order)

  • SUCCESS Python-3.12.3-GCCcore-13.3.0.eb

Build succeeded for 1 out of 1 (1 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/b7839d2f7ab809264c0570e872c91e95 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Oct 22, 2025

Also tested on top of EESSI, see easybuilders/easybuild-easyconfigs#23499 (comment)

Copy link
Copy Markdown
Contributor

@casparvl casparvl left a comment

Choose a reason for hiding this comment

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

Lgtm

@casparvl casparvl merged commit 3fa0df7 into easybuilders:develop Oct 22, 2025
22 checks passed
@boegel boegel deleted the 20251022155908_new_pr_python branch October 22, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug fix EESSI Related to EESSI project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants