Update site package behavior#232

Merged

bedroge merged 24 commits into

EESSI:mainfrom

casparvl:update_site_package_behavior

May 25, 2026

Commits on May 19, 2026

Update site package behavior to allow specifying a custom EESSI_SITE_SOFTWARE_PREFIX, rather than only supporting the default under host_injections
Caspar van Leeuwen
committed
Merge branch 'main' into update_site_package_behavior
Caspar van Leeuwen
committed

Commits on May 22, 2026

use eessi_repo variable instead of os.getenv("EESSI_CVMFS_REPO")
bedroge
authored
Merge branch 'main' into update_site_package_behavior
bedroge
authored
take EESSI_SITE_SOFTWARE_PREFIX into account, and add site's Lmod RC file to LMOD_RC
bedroge
committed
Merge pull request #4 from bedroge/update_site_package_behavior
casparvl
authored
Add proper quoting everywhere. Most of these never went wrong because they are always set (by sourceing early), but it's bad style / practice
Caspar van Leeuwen
committed
Add verbosito for EESSI_SITE_SOFTWARE_PATH
Caspar van Leeuwen
committed
Make sure to also handle the EESSI_SITE_ACCEL_SOFTWARE_PATH
Caspar van Leeuwen
committed
Also make sure to respect the EESSI_SITE_SOFTWARE_PREFIX for the site's accelerator path
Caspar van Leeuwen
committed
Make addition of EESSI_SITE_MODULEPATH_ACCEL conditional on whether the dir exists - just like in the module
Caspar van Leeuwen
committed
string.gsub fails if eessi_module_path_accel is undefined (nil). Guard against that
Caspar van Leeuwen
committed
Guard against eessi_module_path_accel being nil
Caspar van Leeuwen
committed
Try to make the validation pass again: the module now only set the EESSI_SITE_MODULEPATH_ACCEL if that dir exists, so precreate it in the check
Caspar van Leeuwen
committed
Do variables in the correct order when doing variable expansion for replacement...
Caspar van Leeuwen
committed
Fix mistake in testing the existence of the wrong variable
Caspar van Leeuwen
committed
Fix env var name to get the accelerator subdir
Caspar van Leeuwen
committed
Try to figure out why host_injections is a file instead of a dir. Should I just create the /opt/eessi default target?
Caspar van Leeuwen
committed
Let's add -v, to see which dir actually gets created, as we still seem to not get EESSI_SITE_MODULEPATH_ACCEL set
Caspar van Leeuwen
committed
Make sure the full module dir is pre-created
Caspar van Leeuwen
committed
Use EESSI_ACCELERATOR_TARGET, as that gets updated when falling back from e.g. CC77 to CC70
Caspar van Leeuwen
committed
Now actually use the right variable - EESSI_ACCELERATOR_TARGET
Caspar van Leeuwen
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update site package behavior#232

Update site package behavior#232
bedroge merged 24 commits into
EESSI:mainfrom
casparvl:update_site_package_behavior

Commits on May 19, 2026

Update site package behavior to allow specifying a custom EESSI_SITE_SOFTWARE_PREFIX, rather than only supporting the default under host_injections

Merge branch 'main' into update_site_package_behavior

Commits on May 22, 2026

use eessi_repo variable instead of os.getenv("EESSI_CVMFS_REPO")

Merge branch 'main' into update_site_package_behavior

take EESSI_SITE_SOFTWARE_PREFIX into account, and add site's Lmod RC file to LMOD_RC

Merge pull request #4 from bedroge/update_site_package_behavior

Add proper quoting everywhere. Most of these never went wrong because they are always set (by sourceing early), but it's bad style / practice

Add verbosito for EESSI_SITE_SOFTWARE_PATH

Make sure to also handle the EESSI_SITE_ACCEL_SOFTWARE_PATH

Also make sure to respect the EESSI_SITE_SOFTWARE_PREFIX for the site's accelerator path

Make addition of EESSI_SITE_MODULEPATH_ACCEL conditional on whether the dir exists - just like in the module

string.gsub fails if eessi_module_path_accel is undefined (nil). Guard against that

Guard against eessi_module_path_accel being nil

Try to make the validation pass again: the module now only set the EESSI_SITE_MODULEPATH_ACCEL if that dir exists, so precreate it in the check

Do variables in the correct order when doing variable expansion for replacement...

Fix mistake in testing the existence of the wrong variable

Fix env var name to get the accelerator subdir

Try to figure out why host_injections is a file instead of a dir. Should I just create the /opt/eessi default target?

Let's add -v, to see which dir actually gets created, as we still seem to not get EESSI_SITE_MODULEPATH_ACCEL set

Make sure the full module dir is pre-created

Use EESSI_ACCELERATOR_TARGET, as that gets updated when falling back from e.g. CC77 to CC70

Now actually use the right variable - EESSI_ACCELERATOR_TARGET

Commits on May 23, 2026

Use an EESSI_SITE_SOFTWARE_PREFIX for one of the CPU targets to check that this correctly becomes the starting prefix of EESSI_SITE_SOFTWARE_PATH

Add check for starting with EESSI_SITE_SOFTWARE_PREFIX also for the site modulepath and accelerator site modulepath