GNU bug report logs - #57572
29.0.50; Tramp error with tramp-file-name-unify

Previous Next

Package: emacs;

Reported by: Thierry Volpiatto <thievol <at> posteo.net>

Date: Sun, 4 Sep 2022 07:17:01 UTC

Severity: normal

Merged with 57676

Found in version 29.0.50

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: Thierry Volpiatto <thievol <at> posteo.net>
To: Michael Albinus <michael.albinus <at> gmx.de>
Cc: 57572 <at> debbugs.gnu.org
Subject: bug#57572: 29.0.50; Tramp error with tramp-file-name-unify
Date: Sun, 11 Sep 2022 08:50:43 +0000
[Message part 1 (text/plain, inline)]
Hello Michael,

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

> Thierry Volpiatto <thievol <at> posteo.net> writes:
>
> Hi Thierry,
>
>>> But wait, you are in file name completion.
>>
>> Yes, but not sure tramp detects a "file name completion" here as it is
>> not a standard completion used here.
>
> Tramp doesn't need to detect that you're in file name completion or
> whatever, there are some functions to be called depending on the file
> name syntax you pass as arguments, that's all.
>
>>> So I guess you have bound `non-essential' to t. In that case,
>>> `expand-file-name' expands only, if the connection is established
>>> already.
>>
>> I have now bound non-essential to nil _before_ calling
>> file-accessible-directory-p and I have no error.
>> Perhaps it is the right way to do?
>
> non-essential is a mean to tell Tramp, that it shouldn't open a
> connection when it isn't established yet.

Yes, it is what I always understood but I am now confused because the
connection is trigerred by file-accessible-directory-p even with
non-essential==t.


> That's why you bind it in helm to t. Of course it will work when you
> don't bind it.

It is bound to t on top but it is nil in helm-find-files-get-candidates.

> But then, there are other problems.

Yes of course but only if I bind it on top, isn't it?

> The patch I've shown to you yesterday solves your helm-find-files
> case, but the Tramp testsuite fails. I'm analyzing what's up.

Ok thanks.

-- 
Thierry
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 2 years and 255 days ago.

Previous Next


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