Skip to content

Commit e974183

Browse files
casparvlboegel
andauthored
Apply suggestions from code review
Co-authored-by: Kenneth Hoste <[email protected]>
1 parent f237c93 commit e974183

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

create_lmodsitepackage.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@
124124
if packagesList[simpleName] then
125125
-- simpleName is a module in packagesList
126126
-- first, check the old host_injections path prior to https://github.com/EESSI/software-layer-scripts/pull/59
127-
-- If that exists, print a more targetted, explainatory warning
127+
-- If that exists, print a more targetted, explanatory warning
128128
local previousHostInjections = string.gsub(os.getenv('EESSI_SOFTWARE_PATH') or "", 'versions', 'host_injections')
129129
local previousPackageEasyBuildDir = previousHostInjections .. "/software/" .. t.modFullName .. "/easybuild"
130130
local previousPackageDirExists = isDir(previousPackageEasyBuildDir)
@@ -142,7 +142,7 @@
142142
advice = advice .. "due to licencing. You will need to install a full copy of the " .. simpleName .. " package where EESSI "
143143
advice = advice .. "can find it.\\n"
144144
advice = advice .. "Note that a full copy is installed at " .. previousHostInjections .. "/software/" .. t.modFullName .. ". "
145-
advice = advice .. "However, EESSI now expects it in a different location, namely at "
145+
advice = advice .. "However, EESSI expects it in a different location since Aug'25, namely at "
146146
advice = advice .. hostInjections .. "/software/" .. t.modFullName .. ". "
147147
advice = advice .. "Please re-install the package at the new location. "
148148
advice = advice .. refer_to_docs

scripts/gpu_support/nvidia/install_cuda_and_libraries.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ for EASYSTACK_FILE in ${TOPDIR}/easystacks/eessi-*CUDA*.yml; do
132132

133133
# If there is a GPU on the node, the installation path will by default have an
134134
# accelerator subdirectory. For CUDA and cu*, these are binary installations and
135-
# don't care about the target compute capability nor the CPU microarchitecture.
135+
# we don't care about the target compute capability nor the CPU microarchitecture.
136136
# Our hooks are aware of this and therefore expect CUDA to be available under
137137
# something like EESSI_SITE_SOFTWARE_PATH, but then with the CPU micro-architecture
138138
# stripped
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
#!/usr/bin/env bash
22

33
TOPDIR=$(dirname $(realpath $BASH_SOURCE))
4-
echo "This script was replaced by the $TOPDIR/install_cuda_and_libraries.sh script. See https://www.eessi.io/docs/site_specific_config/gpu/ for more information"
4+
echo "This script was replaced by the $TOPDIR/install_cuda_and_libraries.sh script. See https://www.eessi.io/docs/site_specific_config/gpu/ for more information" >&2
5+
exit 1

0 commit comments

Comments
 (0)