-
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
Fix typo in create_lmodsitepackage.py
#696
Conversation
Current typo results in the following problem ```bash [[email protected] ~]$ ml Currently Loaded Modules: 1) StdEnv (S) 2) EESSI/2023.06 Where: S: Module is Sticky, requires --force to unload or purge [ritop@login ~]$ module purge /usr/bin/lua: /cvmfs/software.eessi.io/versions/2023.06/software/linux/x86_64/amd/zen2/.lmod/SitePackage.lua:21: variable 'False' is not declared stack traceback: [C]: in function 'error' /cluster/installations/lmod/lmod/libexec/../tools/strict.lua:36: in function </cluster/installations/lmod/lmod/libexec/../tools/strict.lua:34> /cvmfs/software.eessi.io/versions/2023.06/software/linux/x86_64/amd/zen2/.lmod/SitePackage.lua:21: in function 'from_eessi_prefix' /cvmfs/software.eessi.io/versions/2023.06/software/linux/x86_64/amd/zen2/.lmod/SitePackage.lua:189: in function </cvmfs/software.eessi.io/versions/2023.06/software/linux/x86_64/amd/zen2/.lmod/SitePackage.lua:185> (tail call): ? /cluster/installations/lmod/lmod/libexec/Master.lua:381: in function 'load' /cluster/installations/lmod/lmod/libexec/MasterControl.lua:1009: in function 'load' /cluster/installations/lmod/lmod/libexec/Master.lua:773: in function 'reload_sticky' /cluster/installations/lmod/lmod/libexec/MasterControl.lua:1120: in function 'unload_usr' /cluster/installations/lmod/lmod/libexec/cmdfuncs.lua:496: in function 'Purge' /cluster/installations/lmod/lmod/libexec/cmdfuncs.lua:473: in function 'cmd' /cluster/installations/lmod/lmod/libexec/lmod:512: in function 'main' /cluster/installations/lmod/lmod/libexec/lmod:570: in main chunk [C]: ? ```
Instance
|
Instance
|
Instance
|
bot: build repo:eessi.io-2023.06-software arch:aarch64/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.
lgtm
@ocaisa Looks OK to deploy? |
This is placed in an arch-specific dir so we will need to build for all instances |
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
|
New job on instance
|
New job on instance
|
New job on instance
|
New job on instance
|
New job on instance
|
@boegel Can you trigger a build for Deucalion |
New job on instance
|
New job on instance
|
New job on instance
|
bot: build repo:eessi.io-2023.06-software arch:aarch64/a64fx |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
@boegel Waiting on #696 (comment) and then this is ready to deploy |
all staging PRs merged, so merging this too |
Current typo results in the following problem (found by @TopRichard )