-
Notifications
You must be signed in to change notification settings - Fork 50
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
Rebuild EESSI-extend module to correctly handle EESSI_SITE_INSTALL
#582
Rebuild EESSI-extend module to correctly handle EESSI_SITE_INSTALL
#582
Conversation
Instance
|
Instance
|
bot: build inst:aws repo:eessi.io-2023.06-software arch:zen2 |
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The original behaviour is deliberate so we can create well defined symlinks for the CUDA installation, but maybe I need to rethink this a bit, I guess we could be a bit more loose given we unbreak those links in a very specific way via scripts
This path is also used to immediately expose the site modules via our initialisation scripts |
Should have used EESSI_PROJECT_INSTALL, so this was a misunderstanding on my side. Will close the PR. |
The current module
EESSI-extend/2023.06-easybuild
does not handle the environment settingEESSI_SITE_INSTALL
correctly. It just falls back to using the standardCVMFS
installation path whereversions
is replaced withhost_injections
. This is probably due to an oversight when reviewing PR #371.Note, when rebuilding this for NESSI, we ran into permission issues when EasyBuild tried to set the permissions of the software after it has been built. Will be interesting to see if we run into the same issue here.