GNU bug report logs - #31357
26.1; tramp-make-tramp-file-name: incompatible lisp changes in 26.1 ?

Previous Next

Package: emacs;

Reported by: Phil Sainty <psainty <at> orcon.net.nz>

Date: Thu, 3 May 2018 14:25:02 UTC

Severity: minor

Tags: notabug

Found in version 26.1

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

Bug is archived. No further changes may be made.

Full log


Message #11 received at 31357 <at> debbugs.gnu.org (full text, mbox):

From: Phil Sainty <psainty <at> orcon.net.nz>
To: Michael Albinus <michael.albinus <at> gmx.de>
Cc: 31357 <at> debbugs.gnu.org,
 bug-gnu-emacs <bug-gnu-emacs-bounces+psainty=orcon.net.nz <at> gnu.org>
Subject: Re: bug#31357: 26.1; tramp-make-tramp-file-name: incompatible lisp
 changes in 26.1 ?
Date: Fri, 04 May 2018 09:34:47 +1200
On 2018-05-04 03:26, Michael Albinus wrote:
> Phil Sainty <psainty <at> orcon.net.nz> writes:
>> This should surely be covered in NEWS under the "Incompatible Lisp
>> Changes in Emacs 26.1" section?
> 
> `tramp-make-tramp-file-name' has never been documented as public
> function, it is intended for internal use. Granted, this should have
> been signalled better.

Given the function naming, a NEWS entry seems warranted to me.

I do think that there *should* be a public interface for manipulating
tramp file names.  If these were never intended as such then perhaps
such functionality could be introduced...


> That's the way to go. You could use (funcall 
> 'tramp-make-tramp-file-name ...)
> in order to make the byte compiler quiet.

Perfect.

> However, I wouldn't check for emacs-version. Tramp exist as separate
> package, and the recent Tramp 2.4 runs with all Emacs 24, 25, 26 and
> 27. You might check for tramp-version.

Thanks; I've gone with this:

(apply
 #'tramp-make-tramp-file-name
 (if (version< tramp-version "2.3")
     (list method user host temp hop)
   (list method user domain host port temp hop)))

The changes in 2.4 sound good too.


> (setf (tramp-file-name-host vec) "whatever")

Ah, I'd actually tried that in 25.2 but those are not generalized
variables in that version.  I see that it works in 26.1 though,
so that's an excellent enhancement.


cheers,
-Phil






This bug report was last modified 7 years and 50 days ago.

Previous Next


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