Skip to content

Improve depot management in JuliaPackage easyblock#2935

Merged
migueldiascosta merged 2 commits intoeasybuilders:developfrom
lexming:julia
Jul 20, 2023
Merged

Improve depot management in JuliaPackage easyblock#2935
migueldiascosta merged 2 commits intoeasybuilders:developfrom
lexming:julia

Conversation

@lexming
Copy link
Copy Markdown
Contributor

@lexming lexming commented May 25, 2023

  • fixes easybuild-easyconfigs#17961: user depot will be removed from JULIA_DEPOT_PATH during installation and depot in installdir will be prepended to it
  • partial-fix for easybuild-easyconfigs#17455: paths to depots of loaded modules will be appended to JULIA_DEPOT_PATH on module load to preserve the user depot as top entry in the list

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Jul 17, 2023

Test report by @lexming

Overview of tested easyconfigs (in order)

  • SUCCESS IJulia-1.23.3-Julia-1.6.7.eb
  • SUCCESS IJulia-1.24.0-Julia-1.8.5.eb
  • SUCCESS Julia-1.7.2-linux-x86_64.eb
  • SUCCESS Circuitscape-5.12.3-Julia-1.7.2.eb

Build succeeded for 4 out of 4 (3 easyconfigs in total)
node301.hydra.os - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/lexming/59931ac96b31d28103931632b5976309 for a full test report.

Copy link
Copy Markdown
Member

@migueldiascosta migueldiascosta left a comment

Choose a reason for hiding this comment

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

lgtm

@migueldiascosta
Copy link
Copy Markdown
Member

Going in, thanks @lexming!

@migueldiascosta migueldiascosta merged commit 23e1045 into easybuilders:develop Jul 20, 2023
@lexming lexming deleted the julia branch July 21, 2023 09:54
@boegel boegel changed the title Improve depot management in JuliaPackage Improve depot management in JuliaPackage easyblock Sep 10, 2023
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.

Remove user directory from JULIA_DEPOT_PATH during installation

3 participants