Skip to content

adopt module_load_environment in Java easyblock#3610

Merged
boegel merged 3 commits intoeasybuilders:5.0.xfrom
lexming:modload-java
Feb 18, 2025
Merged

adopt module_load_environment in Java easyblock#3610
boegel merged 3 commits intoeasybuilders:5.0.xfrom
lexming:modload-java

Conversation

@lexming
Copy link
Copy Markdown
Contributor

@lexming lexming commented Feb 17, 2025

Update of Java easyblock for #3527

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Feb 17, 2025

Test report by @lexming

Overview of tested easyconfigs (in order)

  • SUCCESS Java-17.0.6.eb
  • SUCCESS Java-11.0.20.eb
  • SUCCESS Java-21.0.5.eb

Build succeeded for 3 out of 3 (3 easyconfigs in total)
node300.hydra.os - Linux Rocky Linux 8.10 (Green Obsidian), x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 3.9.20
See https://gist.github.com/lexming/503122048fc3c0b48c6de1d2acbaee5d for a full test report.

Comment thread easybuild/easyblocks/j/java.py Outdated
Comment thread easybuild/easyblocks/j/java.py Outdated
…lib directories in post-processing step of Java easyblock
@boegel boegel added the change label Feb 18, 2025
Comment thread easybuild/easyblocks/j/java.py Outdated
Comment thread easybuild/easyblocks/j/java.py Outdated
@boegel boegel changed the title adopt module_load_environment in java easyblock adopt module_load_environment in Java easyblock Feb 18, 2025
@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 18, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Java-11.0.20.eb
  • SUCCESS Java-17.0.6.eb
  • SUCCESS Java-19.0.2.eb
  • SUCCESS Java-21.0.5.eb

Build succeeded for 4 out of 4 (4 easyconfigs in total)
node3514.doduo.os - Linux RHEL 8.8 (Ootpa), x86_64, AMD EPYC 7552 48-Core Processor, Python 3.11.4
See https://gist.github.com/boegel/c950e83173dccac45b2ffc573535a295 for a full test report.

EDIT: this was tested in the EESSI build environment, where both --sysroot and --rpath are used, which is necessary to hit the code block being modified in this PR

@boegel boegel merged commit 161443d into easybuilders:5.0.x Feb 18, 2025
@boegel boegel added the EESSI Related to EESSI project label Feb 18, 2025
@lexming lexming deleted the modload-java branch February 18, 2025 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change EasyBuild-5.0 EasyBuild 5.0 EESSI Related to EESSI project

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants