Message173401
> To be honest I don't really understand the point of the ignore_errors flag on rmtree. If rmtree fails to delete the directory tree (which will happen if one of the files can't be deleted), why would you want it to return succesfully?
I presume it’s meant as a best-effort cleanup.
Regardless both Eric & Serhiy are right: it’s a programmer error to call it on files and it may shadow bugs catching it. OTOH the implementation is inconsistent and not backward compatible now, so we have to fix it unfortunately.
The patch needs to address Giampaolo’s (bug tracker) & Serhiy’s (Rietveld) comments before it can be merged though – thanks. :) |
|
Date |
User |
Action |
Args |
2012-10-20 17:48:37 | hynek | set | recipients:
+ hynek, barry, pitrou, larry, giampaolo.rodola, jwilk, eric.araujo, chris.jerdonek, serhiy.storchaka, eng793 |
2012-10-20 17:48:36 | hynek | link | issue15872 messages |
2012-10-20 17:48:36 | hynek | create | |
|