Thanks to visit codestin.com
Credit goes to github.com

Skip to content

run chmod -R a+rX $(STORE) before creating sqfs#206

Merged
bcumming merged 1 commit into
masterfrom
chmodarX
Jan 16, 2025
Merged

run chmod -R a+rX $(STORE) before creating sqfs#206
bcumming merged 1 commit into
masterfrom
chmodarX

Conversation

@simonpintarelli

Copy link
Copy Markdown
Member

Should we run chmod -R a+rX $(STORE) before creating the squashfs file?

Currently chmod needs to be run manually in post-install. User wouldn't need to think about this when writing post-install hooks.

Ping @lukasgd.

@bcumming

Copy link
Copy Markdown
Member

Naive question: what is the reason for this?

@simonpintarelli

Copy link
Copy Markdown
Member Author

For example when manually installing packages in post-install, it is likely one needs to add chmod, since the files are owned by root and not world-readable:
https://github.com/biddisco/alps-spack-stacks/blob/paraview/recipes/paraview/5.13.2/gh200/post-install#L208

Having everything in $(STORE) world-readable seems a sane default. Although currently it is mostly redundant, since we take care of permissions for the bulk of the files in other places.

@bcumming bcumming merged commit 701ccf8 into master Jan 16, 2025
@simonpintarelli simonpintarelli deleted the chmodarX branch January 16, 2025 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants