Message342051
Patch LGTM. I'd like to point out one thing for posterity. Current shutil code catches EPERM but not EACCES. While reviewing the patch I wondered whether simply catching (and ignoring) both error codes instead, but it turns out they are supposed to have 2 different meanings:
https://stackoverflow.com/a/35879961/376587
Especially after _copyxattr is moved before chmod() EACCES (permission denied) should propagate instead of being silenced, and EPERM should be left in place because in this case EPERM acts more like an ENOTSUP (operation not supported) than EACCES (permission denied) and as such should rightly be ignored. |
|
Date |
User |
Action |
Args |
2019-05-10 03:20:02 | giampaolo.rodola | set | recipients:
+ giampaolo.rodola, tarek, r.david.murray, Gerrit.Holl, Bart Oldeman, Maxime Boissonneault, obilaniu |
2019-05-10 03:20:02 | giampaolo.rodola | set | messageid: <1557458402.21.0.915825106263.issue24538@roundup.psfhosted.org> |
2019-05-10 03:20:02 | giampaolo.rodola | link | issue24538 messages |
2019-05-10 03:20:02 | giampaolo.rodola | create | |
|