[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: tramp (; setting GZIP environment variable breaks tramp

From: Francesco Potortì
Subject: Re: tramp (; setting GZIP environment variable breaks tramp
Date: Fri, 26 Oct 2018 18:36:33 +0200

>> $ echo | gzip -c >/dev/null 2>/dev/null
>> $ echo | env --unset=GZIP gzip -c >/dev/null
>> So one of these would be a simple solution.  But I wonder: since tramp
>> checks for a working compression program, why doesn't it realise right
>> from the start that gzip does not work?  I think it should.
>Tramp checks only the remote gzip command, and it uses >/dev/null
>redirection there. Your problem seems to be the local gzip call.

Yes, it's the local one.  Anyway >/dev/null changes nothing, the warning
is on stderr

>Whant happens, if you add "GZIP" to process-environment (without the
>"=VALUE" part)?

That will do the trick.  It should be changed locally with a let binding.

>Alternatively, you could change the first entry in
>tramp-inline-compress-commands to
>  '(("env GZIP= gzip" "env GZIP= gzip -d")


>(I don't want to use --unset; I'm not sure that all incarnations of env
>do understand this.)

I see.

>Does either of this work?

I haven't tried either on elisp, because in the meantime I have changed
my environment and removed the GZIP env var.  But from shell experiments
they should work:

$ echo | gzip -c >/dev/null
gzip: warning: GZIP environment variable is deprecated; use an alias or script

$ echo | GZIP= gzip -c >/dev/null

$ echo | env GZIP= gzip -c >/dev/null

If you decide on either and send me a patch I can try that.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]