[issue34969] Add --fast, --best to the gzip CLI

2018-11-03 Thread Julien Palard
Change by Julien Palard : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue34969] Add --fast, --best to the gzip CLI

2018-11-03 Thread Julien Palard
Julien Palard added the comment: New changeset 3e28eed9ec2249bb11ad0db4629271b7ce9b7918 by Julien Palard (Stéphane Wirtel) in branch 'master': bpo-34969: Add --fast, --best on the gzip CLI (GH-9833) https://github.com/python/cpython/commit/3e28eed9ec2249bb11ad0db4629271b7ce9b7918 --

[issue34969] Add --fast, --best to the gzip CLI

2018-10-29 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: Hi @Serhiy I would like to add them on lzma, bz2, zipfile and tarfile. -- ___ Python tracker ___

[issue34969] Add --fast, --best to the gzip CLI

2018-10-29 Thread STINNER Victor
STINNER Victor added the comment: man lzma: --fast --best These are somewhat misleading aliases for -0 and -9, respec‐ tively. These are provided only for backwards compatibility with LZMA Utils. Avoid using these options. man bzip2:

[issue34969] Add --fast, --best to the gzip CLI

2018-10-29 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: If we add this options to the gzip CLI, I would suggest to add them to bzip2 and lzma CLI in the same PR. And maybe open separate issues for the zipfile and tarfile CLI, -- ___ Python tracker

[issue34969] Add --fast, --best to the gzip CLI

2018-10-29 Thread STINNER Victor
STINNER Victor added the comment: > But if I add options for controlling the compression level, I would add > options -1 ... -9. I never used verbose forms --fast and --best. If we add options, I would prefer to only add --fast and --best which are easy to understand. I really have no idea

[issue34969] Add --fast, --best to the gzip CLI

2018-10-20 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: I'm not sure those options are such useful. If you want to control more details, it is not hard to write a tiny Python script. Python is a programming language on the whole. But if I add options for controlling the compression level, I would add options

[issue34969] Add --fast, --best to the gzip CLI

2018-10-13 Thread Stéphane Wirtel
Stéphane Wirtel added the comment: Julien, Currently, the default compresslevel for gzip.open, GzipFile is 9, for the best method compression. Maybe we could define 6 as the tradeoff and specify 1 (for fast), 9 (for best), 6 (tradeoff). Maybe in an other issue? --

[issue34969] Add --fast, --best to the gzip CLI

2018-10-13 Thread pmpp
pmpp added the comment: Hi, on platform without gzip ( there are some , including some widely used OS eg: https://github.com/bazelbuild/rules_docker/issues/507 ) ability to use python gzip cli is extremely usefull as a fallback. Though as discussed on irc default compression to 6 is a good

[issue34969] Add --fast, --best to the gzip CLI

2018-10-13 Thread Julien Palard
Julien Palard added the comment: Hi Stéphane thanks for the proposal and the PR. But are those options usefull in real life? (I may be biased as a Linux user). I see this gzip CLI usefull to decompress a gzip file on platforms not having a gzip program installed, but I don't think it's

[issue34969] Add --fast, --best to the gzip CLI

2018-10-13 Thread Stéphane Wirtel
Change by Stéphane Wirtel : -- nosy: +mdk stage: patch review -> ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue34969] Add --fast, --best to the gzip CLI

2018-10-12 Thread Stéphane Wirtel
Change by Stéphane Wirtel : -- keywords: +patch pull_requests: +9207 stage: -> patch review ___ Python tracker ___ ___

[issue34969] Add --fast, --best to the gzip CLI

2018-10-12 Thread Stéphane Wirtel
New submission from Stéphane Wirtel : the gzip module has a CLI but this one does not allow to specify the compression method (slow, fast) -- assignee: matrixise messages: 327626 nosy: matrixise priority: normal severity: normal status: open title: Add --fast, --best to the gzip CLI