Created on 2009-05-18 05:01 by joe.amenta, last changed 2022-04-11 14:56 by admin. This issue is now closed.
Files |
|
|
|
File name |
Uploaded |
Description |
Edit |
zipdiff.diff |
joe.amenta,2009-05-18 05:01 |
extract() tests if dir exists, does not try to os.mkdir if it already does. |
|
Messages (4) |
|
|
msg88014 - (view) |
Author: Joe Amenta (joe.amenta) |
Date: 2009-05-18 05:01 |
zipfile.ZipFile.extract() fails if targetpath is a directory that already exists. Bug revealed itself on Ubuntu, using extractall(). Happened on the latest 3.1 and 2.7. Attached a patch that I think will fix this issue. |
|
|
msg88243 - (view) |
Author: Benjamin Peterson (benjamin.peterson) *  |
Date: 2009-05-23 17:10 |
It would be nice to have a test with the patch. |
|
|
msg88277 - (view) |
Author: Martin v. Löwis (loewis) *  |
Date: 2009-05-24 19:55 |
Thanks for the patch. Committed with modifications (including a test) as r72893, r72894, r72895, r72896. |
|
|
msg111962 - (view) |
Author: Ezio Melotti (ezio.melotti) *  |
Date: 2010-07-29 16:35 |
#9172 has been closed as duplicate of this. |
|
|
History |
|
|
|
Date |
User |
Action |
Args |
2022-04-11 14:56:49 |
admin |
set |
github: 50300 |
2010-07-29 16:35:15 |
ezio.melotti |
set |
nosy: + ezio.melottimessages: + type: behaviorstage: resolved |
2009-05-24 19:55:45 |
loewis |
set |
status: open -> closedresolution: acceptedmessages: + |
2009-05-23 17:10:37 |
benjamin.peterson |
set |
nosy: + benjamin.petersonmessages: + |
2009-05-18 06:14:51 |
loewis |
set |
priority: release blockerassignee: loewisnosy: + loewis |
2009-05-18 05:01:40 |
joe.amenta |
create |
|