Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Back out "github-runner: replace mkdir -p -m with umask" #1386

Merged
merged 1 commit into from
Mar 16, 2025

Conversation

Enzime
Copy link
Collaborator

@Enzime Enzime commented Mar 16, 2025

This backs out commit 3b738c7.

Setting a umask made the parent directory have too conservative of permissions making it so _github-runner couldn't access the child directories.

This backs out commit 3b738c7.

Setting a `umask` made the parent directory have too conservative of
permissions making it so `_github-runner` couldn't access the child
directories.
@Enzime Enzime force-pushed the push-lnuotyxsytkz branch from ba47db2 to 7b4a495 Compare March 16, 2025 10:31
Copy link
Collaborator

@emilazy emilazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’ll trust you :)

@Enzime Enzime merged commit 95eac71 into LnL7:master Mar 16, 2025
3 checks passed
@Enzime Enzime deleted the push-lnuotyxsytkz branch March 16, 2025 22:58
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