Update site package behavior#232

Open

casparvl wants to merge 19 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
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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update site package behavior#232

Update site package behavior#232
casparvl wants to merge 19 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