Author nadeem.vawda
Recipients eric.araujo, nadeem.vawda, proyvind, tarek
Date 2012-02-11.18:28:32
SpamBayes Score 0.00471207
Marked as misclassified No
Message-id <1328984913.38.0.797072619928.issue5411@psf.upfronthosting.co.za>
In-reply-to
Content
> This not-so-bad patch adds lzma compression support to the shutil functions, under the 'xztar' name.  Please review.

Functionally, the patch looks good to me. There are some docstrings that
should probably be updated, though:

- _make_tarball (the 'compress' argument)
- make_archive
- _unpack_tarfile
- unpack_archive


> Ideally, tarfile should have an attribute to expose what compression formats are available, then shutil would reuse that, and packaging would just call shutil.

+1
History
Date User Action Args
2012-02-11 18:28:33nadeem.vawdasetrecipients: + nadeem.vawda, tarek, eric.araujo, proyvind
2012-02-11 18:28:33nadeem.vawdasetmessageid: <1328984913.38.0.797072619928.issue5411@psf.upfronthosting.co.za>
2012-02-11 18:28:32nadeem.vawdalinkissue5411 messages
2012-02-11 18:28:32nadeem.vawdacreate