desihub / desisurvey

Code for desi survey planning and implementation
BSD 3-Clause "New" or "Revised" License
2 stars 7 forks source link

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

Open schlafly opened 1 year ago

schlafly commented 1 year ago

The NTS move_tile_into_place routine needs to have an additional chmod command to make newly created files group writeable here https://github.com/desihub/desisurvey/blob/7200158600d4a9702341b469e695766f489da855/py/desisurvey/NTS.py#L230-L231 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.