Skip to content

replace deprecated expand_module_search_path with ModuleEnvironmentVariable.expand_paths() in Bundle and Java easyblocks#3727

Merged
boegel merged 3 commits intoeasybuilders:developfrom
lexming:migrate-expand-paths
May 22, 2025
Merged

replace deprecated expand_module_search_path with ModuleEnvironmentVariable.expand_paths() in Bundle and Java easyblocks#3727
boegel merged 3 commits intoeasybuilders:developfrom
lexming:migrate-expand-paths

Conversation

@lexming
Copy link
Copy Markdown
Contributor

@lexming lexming commented May 22, 2025

No description provided.

@lexming lexming added the update label May 22, 2025
@lexming lexming added this to the next release (5.1.0) milestone May 22, 2025
@boegel boegel changed the title replace deprecated expand_module_search_path with ModuleEnvironmentVariable.expand_paths() in Bundle and Java easyblocks replace deprecated expand_module_search_path with ModuleEnvironmentVariable.expand_paths() in Bundle and Java easyblocks May 22, 2025
@boegel
Copy link
Copy Markdown
Member

boegel commented May 22, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Java-8.452.eb
  • SUCCESS Java-11.0.27.eb
  • SUCCESS Java-17.0.15.eb
  • SUCCESS Java-21.0.7.eb

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

edit: tested in EESSI build environment, since changed code block in Java easyblock is only relevant when sysroot EasyBuild configuration setting is used;

@boegel
Copy link
Copy Markdown
Member

boegel commented May 22, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS FASTX-Toolkit-0.0.14-GCC-11.3.0.eb
  • SUCCESS Xvfb-21.1.9-GCCcore-13.2.0.eb
  • SUCCESS FlexiBLAS-3.4.4-GCC-13.3.0.eb
  • SUCCESS X11-20240607-GCCcore-13.3.0.eb

Build succeeded for 4 out of 4 (4 easyconfigs in total)
node3505.doduo.os - Linux RHEL 9.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.9.18
See https://gist.github.com/boegel/6926544aa449c98a39d11a44b89da3ef for a full test report.

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 boegel merged commit 179c265 into easybuilders:develop May 22, 2025
17 checks passed
@boegel boegel added the bug fix label May 22, 2025
@lexming lexming deleted the migrate-expand-paths branch May 23, 2025 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants