Fix permission issue for newly created dirs/files in install_scripts.sh
#38
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This issue surfaced when building the init dir for the new 2025.06 version, and only on Deucalion: the permissions on the
init/bash
andmodules/EESSI/2025.lua
files were wrong, and didn't have read permissions for others. I suspect it's related to the default umask on Deucalion (?).I noticed that
install_scripts.sh
already used a special procedure insed_update_if_changed
(see https://github.com/EESSI/software-layer-scripts/blob/main/install_scripts.sh#L58), which explicitly sets the umask and usescat
for copying the files (apparently to retain the permissions?). This PR uses the same approach for copying missing/modified files (instead of usingcp
). #37 (comment) shows that this solves the issue on Deucalion.