On 2025-02-12 05:58, Lasse Collin wrote:

> the removal makes gzip one
> of the few compression tools that don't support setting even the
> compression level via an environment variable.

Thanks for the review. On further thought I agree that, for backward 
compatibility reasons at least, the gzip command should continue to 
examine the GZIP environment variable for options like -9 that affect 
only performance (including compression performance). These options 
shouldn't cause the bigger glitches (and perhaps even security hazards) 
that behavior-oriented options can cause. This would be more in line 
with zstd, which takes this relatively-cautious approach to environment 
variables.

You mentioned --synchronous as being innocuous even though gzip 
currently doesn't allow it in GZIP, and that also is a good suggestion.

So I installed the attached patch instead into Savannah master gzip. It 
means gzip will pay attention to -1..-9, --rsyncable, and --synchronous 
but will silently ignore everything else in GZIP. The idea is that a 
garbage GZIP value shouldn't disrupt normal operations.