[issue2846] Gzip cannot handle zero-padded output + patch

Antoine Pitrou added the comment: Thank you Brian. I've committed the patch into trunk and py3k. I haven't backported it to 2.6 and 3.1, since it's more a new feature than a bug fix. -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed

[issue2846] Gzip cannot handle zero-padded output + patch

Brian Curtin added the comment: Thanks for taking a look! Patch updated with that try/except removed. -- Added file: http://bugs.python.org/file15856/issue2846.diff ___ Python tracker __

[issue2846] Gzip cannot handle zero-padded output + patch

Changes by Brian Curtin : Removed file: http://bugs.python.org/file15852/issue2846.diff ___ Python tracker ___ ___ Python-bugs-list mailing lis

[issue2846] Gzip cannot handle zero-padded output + patch

Antoine Pitrou added the comment: There is no need to write: try: [...] except IOError as err: self.fail(err) Just let the exception be raised and produce an error. -- nosy: +pitrou ___ Python tracker

[issue2846] Gzip cannot handle zero-padded output + patch

Brian Curtin added the comment: Updated patch with some documentation -- Added file: http://bugs.python.org/file15852/issue2846.diff ___ Python tracker ___ __

[issue2846] Gzip cannot handle zero-padded output + patch

Changes by Brian Curtin : Removed file: http://bugs.python.org/file15851/issue2846.diff ___ Python tracker ___ ___ Python-bugs-list mailing lis

[issue2846] Gzip cannot handle zero-padded output + patch

Brian Curtin added the comment: Here tadek's patch updated for trunk, with a test added to it. I feel like this should be documented somewhere, but Doc/Library/gzip.rst doesn't feel right. Maybe it just needs a mention in the "What's new" or something? -- assignee: -> brian.curtin

[issue2846] Gzip cannot handle zero-padded output + patch

New submission from Tadek Pietraszek <[EMAIL PROTECTED]>: There are cases when gzip produces/receives a zero-padded output, for example when creating a compressed tar archive with a pipe: tar cz /dev/null > foo.tgz ls -la foo.tgz -rw-r- 1 tadek tadek 10240 May 13 23:40 foo.tgz tar tvfz foo