-
Notifications
You must be signed in to change notification settings - Fork 720
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
set JULIA_DEPOT_PATH in Julia easyconfigs with default value #17976
set JULIA_DEPOT_PATH in Julia easyconfigs with default value #17976
Conversation
@boegelbot: please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1563049751 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ jsc-zen2 |
@lexming: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1564258426 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Hi all, One point to discuss: do we really want to point to the "shared" julia depots? Since 1.9.0 Julia will actually recompile much of the base packages anyway (depending on dependency updates etc), so I'd say that any attempts to push a package directory to it might be completely vain. The other concern is that the And one note: I'm not sure if ~ in the configs here works as intended (especially for the history dir -- see the other PR). Thanks! |
~/.julia
as top entry in $JULIA_DEPOT_PATH
Conflicts: easybuild/easyconfigs/j/Julia/Julia-1.9.0-linux-x86_64.eb
~/.julia
as top entry in $JULIA_DEPOT_PATH
@exaexa I checked your PR #17995 and as I commented there, entirely unsetting I agree with you that using Let me know if this approach works for you. |
@boegelbot: please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1626357996 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
…asyconfigs into 20230525163634_new_pr_Julia161
@boegelbot: please test @ generoso |
@lexming: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1637704075 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ jsc-zen2 |
@lexming: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1637739029 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @branfosj |
Going in, thanks @lexming! |
(created using
eb --new-pr
)As reported in #17455, user depot should be the top entry in
JULIA_DEPOT_PATH
. This PR fixes that issue by simply settingJULIA_DEPOT_PATH
to:
, which is the default value with user's depot as top entry.Having
JULIA_DEPOT_PATH
defined in the base Julia easyconfigs is needed for the changes in easybuilders/easybuild-easyblocks#2935 to fix #17961