Test Results

 30 files   - 18  30 suites   - 18   37s ⏱️ -11s
 54 tests  - 15  50 ✅  - 12  4 💤  - 3  0 ❌ ±0 
102 runs   - 45  96 ✅  - 36  6 💤  - 9  0 ❌ ±0 

Results for commit 2898922. ± Comparison against base commit 7439bae.

magpie ‑ [magpie] Verify MAGPIE module is loaded and matches rpm version
magpie ‑ [magpie] Verify module MAGPIE_DIR is defined and exists
magpie ‑ [magpie] check for RPM
mem_limits ‑ [memlock] check increased hard limit
mem_limits ‑ [memlock] check increased soft limit
munge ‑ [munge] Decode credential locally
munge ‑ [munge] Generate a credential
munge ‑ [munge] Run benchmark
munge ‑ [munge] check for OS provdied RPM
ompi_info ‑ [openmpi] check for no output to stderr with ompi_info
…

lmod ‑ [lmod] test that the setup function passed

I don't think the .tar.gz#$ is really needed, but I can re-add if the change breaks your workflow. It's presence broke mine.

Let's see if it works in OBS. Not sure why it exists.

I still think it's not great to unconditionally append : to MANPATH - what if it was already present?

See #2070 and #2105. Initially it was checking if the colon exists at the end, but that wasn't working correctly for all use cases. Not sure, maybe the check was not working correctly. So I didn't want to completely undo the change because having the colon at the end made sense. If you have a better solution, please provide a patch. Maybe with your changes concerning escaping $ it would now work better.

The build system is happy with your changes: https://obs.openhpc.community/package/show/OpenHPC3:3.3.1:Factory/lmod