Code review comment for lp:~andreserl/maas/maas_set_correct_file_permissions

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Ah, I rubbed the sleep out of my eyes and took a better look. I was completely mistaken about the second issue -- although I still can't figure out what the basis for the assumption is. I suspect that chmod inside atomic_write may belong in the caller.

« Back to merge proposal