-
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
Put a Lmod-relevant wrapper in place for archdetect accelerator detection #783
Put a Lmod-relevant wrapper in place for archdetect accelerator detection #783
Conversation
Instance
|
Instance
|
Instance
|
bot: build repo:eessi.io-2023.06-software arch:x86_64/generic |
Updates by the bot instance
|
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.
Looks good to me. Maybe a bit explanation of why this is necessary and how it would be used (e.g., alone or together with lmod_eessi_archdetect_wrapper.sh
) would help to understand/assess it more easily.
Label |
This is part of #781 , where I have a chicken egg situation (I need the script in place so the module can be tested in CI). The wrapper is required so I only take absolutely minimal changes from the called script (in my case I just want the value of result, no error codes or anything) |
Script has become available via CernVM-FS:
|
PR merged! Moved |
PR merged! Moved |
PR merged! Moved |
No description provided.