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

Change permissions on newly created files to be group writeable #155

Open
schlafly opened this issue Oct 1, 2023 · 0 comments
Open

Change permissions on newly created files to be group writeable #155

schlafly opened this issue Oct 1, 2023 · 0 comments

Comments

@schlafly
Copy link
Contributor

schlafly commented Oct 1, 2023

The NTS move_tile_into_place routine needs to have an additional chmod command to make newly created files group writeable here

os.makedirs(os.path.dirname(os.path.join(fadir, fabasefn)),
exist_ok=True, mode=0o2775)

This looks like it should be doing the right thing, but the umask is applied on top of the mode, so while the mode looks like it wants 775, that does not end up being the actual permissions on the directory. An explicit chmod addresses the issue.

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

No branches or pull requests

1 participant