This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author r.david.murray
Recipients KevinH, alanmcintyre, ned.deily, r.david.murray, xuanji
Date 2010-12-20.01:22:09
SpamBayes Score 3.0662695e-12
Marked as misclassified No
Message-id <1292808131.21.0.324027325657.issue10694@psf.upfronthosting.co.za>
In-reply-to
Content
Actually our normal procedure currently (this will change a bit after the migration to mercurial) is a patch against the py3k branch, and the committer will do the backport to the other active branches.  If the 2.7 code is very different, a separate 2.7 patch is sometimes helpful.  In this case it looks like a patch that applies to py3k is enough.

Alan hasn't spoken up yet, and he's listed as maintainer.  If he doesn't speak up someone else will hopefully make a decision before the end of the beta period for 3.2.  I'm bumping up the priority of this issue because I think not being able to handle commonly produced archives that other zip tools can handle is a relatively serious defect.

My own opinion is that truncation if the extra data doesn't look like a comment makes the most sense.  But absent an opinion from Alan I think I'd be guided by whatever other zip tools do in this case.  Kevin, am I correct in guessing that that is truncation?

Ned is definitely correct that absence of a test case is something that can delay getting a patch applied.  If a patch includes a unit test, a committer can often do a quick review-and-apply, while absent a unit test the committer would first need to write one, and therefore will often move on to some other, easier to process issue :)  In this case it seems to me that the unit test will only differ in one detail depending on the fix chosen, so it may be worth writing it even before a final decision is made, if you are willing.
History
Date User Action Args
2010-12-20 01:22:11r.david.murraysetrecipients: + r.david.murray, alanmcintyre, ned.deily, xuanji, KevinH
2010-12-20 01:22:11r.david.murraysetmessageid: <1292808131.21.0.324027325657.issue10694@psf.upfronthosting.co.za>
2010-12-20 01:22:09r.david.murraylinkissue10694 messages
2010-12-20 01:22:09r.david.murraycreate