Sean Rodman added the comment:
Thank you for reviewing it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19614
___
___
Python-bugs-list mailing
Roundup Robot added the comment:
New changeset a5f767bf9d1c by Benjamin Peterson in branch '3.3':
use support.rmtree instead of shutil (closes #19614)
http://hg.python.org/cpython/rev/a5f767bf9d1c
New changeset 8a77e22aff6c by Benjamin Peterson in branch 'default':
merge 3.3 (#19614)
R. David Murray added the comment:
Thanks. Yes, the fix is in temp_dir. Patch looks fine, may be a bit before I
get around to applying it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19614
Sean Rodman added the comment:
Hey r.david.murray, so should this change be made in the test?
--
nosy: +sean.rodman
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19614
___
Sean Rodman added the comment:
I don't see where temp_cwd uses shutil.rmtree, but I do see where temp_dir uses
shutil.rmtree. Here is a patch to change that to support.rmtree. If I am way
off base on this patch please let me know and I will change it to fix whatever
needs to be fixed. I want
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +ezio.melotti, pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19614
___
___
New submission from R. David Murray:
Based on this error on one of the buildbots, it is clear that support.temp_cwd
should be calling support.rmtree, and not shutil.rmtree, during cleanup:
[...]
During handling of the above exception, another exception occurred:
Traceback (most recent call