New submission from Zhigang Wang :
We only support arcname with one leading '/', but not more. This patch fixes it.
We don't support arcname with '..' well. The default behavior of unzip and 7z
is to ignore all '..'. This patch does the same.
Also updated t
Zhigang Wang added the comment:
$ unzip -l t.zip
Archive: t.zip
Length DateTimeName
- -- -
3 01-14-2011 21:11 ../foo
3 01-14-2011 21:11 foo
- ---
6 2 files
[zhigang
Zhigang Wang added the comment:
Yes, in zipfile, we just overwrite it. Actually, ZipFile.extract() overwrite
existing files already. If we want it more powerful, we can add a 'overwrite'
parameter. But turning zipfile a full featured zip/unzip tool needs much more
New submission from Zhigang Wang :
The current subprocess.Popen() has a boolean close_fds parameter, which
cannot satisfy all the requirements. Eg. want to pass specific fd to
child process, but close others.
This patch adds a extra parameter pass_fds to subprocess.Popen's
__init__().
Zhigang Wang added the comment:
Thanks Robert for pointing out issue553171.
After read that issue, I still think we paid too much to make shelf less
surprising.
We should at lease provide a option for the *smart* programmers to get
better speed and less exceptions.
The write-back-all-cache
New submission from Zhigang Wang :
shelf without writeback=True is too slow, while shelves with
writeback=True takes too much time to close. And even worse, these codes
can not run:
$ cat test_shelve.py
#!/usr/bin/env python
import shelve
store = shelve.open("/tmp/shelve.db", writ
Zhigang Wang added the comment:
Add some errata of the patch: add the new class to __all__.
--
Added file: http://bugs.python.org/file13318/fast_shelf-v2.patch
___
Python tracker
<http://bugs.python.org/issue5
