[issue5950] Make zimport work with zipfile containing comments

2010-07-11 Thread Ryan Kelly
Changes by Ryan Kelly r...@rfk.id.au: -- nosy: +rfk ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5950 ___ ___ Python-bugs-list mailing list

[issue5950] Make zimport work with zipfile containing comments

2010-07-11 Thread Ryan Kelly
Ryan Kelly r...@rfk.id.au added the comment: Attached is my attempt at a patch for this functionality, along with some simple tests. This basically mirrors what's done in zipfile.py, searching backwards through the file until it finds the end-of-central-directory marker. It tries to be

[issue5950] Make zimport work with zipfile containing comments

2010-07-11 Thread Ryan Kelly
Ryan Kelly r...@rfk.id.au added the comment: Whoops, forgot to remove the line from the docs about comments not being supported. Updated the patch accordingly. -- Added file: http://bugs.python.org/file17944/zipimport_with_comments.patch ___ Python

[issue5950] Make zimport work with zipfile containing comments

2010-07-11 Thread Ryan Kelly
Changes by Ryan Kelly r...@rfk.id.au: Removed file: http://bugs.python.org/file17943/zipimport_with_comments.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5950 ___

[issue5950] Make zimport work with zipfile containing comments

2010-07-09 Thread Dmitry
Dmitry d...@samersoff.net added the comment: I'm talking about internal zimport function (see attached testcase): i.e. import sys; sys.path.insert(0,'test.zip'); import test test.testme() doesn't work if test.zip contains comment. -- ___ Python

[issue5950] Make zimport work with zipfile containing comments

2010-07-08 Thread Terry J. Reedy
Terry J. Reedy tjre...@udel.edu added the comment: Does 'zimport' refer to the 3.1 zipfile or zipimport modules or an internal function called zimport? -- nosy: +tjreedy title: zimport doesn't work with zipfile containing comments - Make zimport work with zipfile containing comments