Skip to content

Implement support for the umask and umask_override defaults #1151

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 27, 2025
Merged

Implement support for the umask and umask_override defaults #1151

merged 3 commits into from
Jun 27, 2025
+130 −3

Conversation

bjorn3
Copy link
Collaborator

@bjorn3 bjorn3 commented Jun 6, 2025 •
edited
Loading

Fixes #1118

@bjorn3 bjorn3 marked this pull request as ready for review June 6, 2025 14:47
@bjorn3 bjorn3 requested a review from squell June 6, 2025 14:47
@squell
Copy link
Member

squell commented Jun 27, 2025 •
edited
Loading

Note: this does not respect a umask that has been set using a per-command Defaults (this is a simple fix I think)

@squell squell force-pushed the umask branch 2 times, most recently from 12024bf to 5f4aecb Compare June 27, 2025 14:37
@squell squell enabled auto-merge June 27, 2025 14:37
@squell squell merged commit ba383eb into trifectatechfoundation:main Jun 27, 2025
15 checks passed
@bjorn3 bjorn3 deleted the umask branch June 27, 2025 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@squell squell squell approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support umask
2 participants
@bjorn3 @squell