GNU bug report logs - #63300
29.0.90; Tramp keeps customizing variables on every connection

Previous Next

Package: emacs;

Reported by: Jimmy Yuen Ho Wong <wyuenho <at> gmail.com>

Date: Fri, 5 May 2023 10:43:02 UTC

Severity: normal

Tags: patch

Found in version 29.0.90

Fixed in version 29.1

Done: Michael Albinus <michael.albinus <at> gmx.de>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Jimmy Wong <wyuenho <at> gmail.com>
To: Michael Albinus <michael.albinus <at> gmx.de>
Cc: 63300 <at> debbugs.gnu.org
Subject: bug#63300: 29.0.90; Tramp keeps customizing variables on every connection
Date: Sun, 4 Jun 2023 17:55:42 +0100
[Message part 1 (text/plain, inline)]
Actually, I take it back. customize-save-variable only saves variables that have the saved-value symbol property set. This property is normally set by customize when it is saved to the custom file. Something in Tramp is setting this property without saving the variable values to the custom file or going through customize. In fact, you can see it via M-x customize-option connection-local-criteria-alist. The value will be shown as SAVED.

In addition, setting `enable-connection-local-variables` to nil early in early-init.el has no effect, the variable is still set on load.

Both of this behavior violate what is documented in the docstring and https://www.gnu.org/software/emacs/manual/html_node/elisp/Standard-Properties.html.

If you must, enable-conneciton-local-variables should be respected, and if you must, set the theme-value property on these connection-local variables instead of saved-value so custom-save-all will not save them.

On 4 Jun 2023 at 1:23 PM +0100, Jimmy Wong <wyuenho <at> gmail.com>, wrote:
> I’ll file a separate bug. You can close this one. Thanks.
> On 4 Jun 2023 at 1:21 PM +0100, Michael Albinus <michael.albinus <at> gmx.de>, wrote:
> > Jimmy Wong <wyuenho <at> gmail.com> writes:
> >
> > Hi,
> >
> > > Well, yes, I consider this a poorly considered “feature”, or arguably
> > > a bug, for exactly the reason you said, they are local to connections
> > > and machines, and thus highly variable.
> >
> > Not really. They exist for the reason that a user can override the
> > default values offered by Tramp. This I would call a user option.
> >
> > Tramp has a lot of further host specific data, which are all kept in
> > variables.
> >
> > > We can probably argue whether this is a bug in tramp or a bug in
> > > customize-save-variable tho (why on earth does the act of saving a
> > > single variable value saves all customized values if a custom file
> > > exists?)
> >
> > I agree with the latter.
> >
> > Best regards, Michael.
[Message part 2 (text/html, inline)]

This bug report was last modified 1 year and 353 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.