From: Michael Albinus <michael.albinus@gmx.de>
To: Eli Zaretskii <eliz@gnu.org>
Cc: vidagabor@gmail.com, 14030@debbugs.gnu.org
Subject: bug#14030: 24.3; expand-file-name expands remote file name on Windows incorrectly
Date: Sun, 24 Mar 2013 18:21:48 +0100 [thread overview]
Message-ID: <87620g67df.fsf@gmx.de> (raw)
In-Reply-To: <83r4j43hiz.fsf@gnu.org> (Eli Zaretskii's message of "Sun, 24 Mar 2013 18:10:44 +0200")
Eli Zaretskii <eliz@gnu.org> writes:
> My crystal ball says that your 24.3.50 snapshot was bootstrapped on
> MS-Windows, or at least Tramp was compiled on Windows. See below.
In GNU Emacs 24.3.50.1 (i386-mingw-nt6.1.7600)
of 2013-03-14 on VBOX
Bzr revision: 112044 eliz@gnu.org-20130314173751-20hpndyud1tka138
Windowing system distributor `Microsoft Corp.', version 6.1.7600
Configured using:
`configure --with-gcc (4.7) --no-opt --enable-checking --cflags
-IC:/emacs/libs/libXpm-3.5.10/include -IC:/emacs/libs/libXpm-3.5.10/src
-IC:/emacs/libs/libpng-dev_1.4.3-1_win32/include
-IC:/emacs/libs/zlib-dev_1.2.5-2_win32/include
-IC:/emacs/libs/giflib-4.1.4-1-lib/include
-IC:/emacs/libs/jpeg-6b-4-lib/include
-IC:/emacs/libs/tiff-3.8.2-1-lib/include
-IC:/emacs/libs/libxml2-2.7.8-w32-bin/include/libxml2
-IC:/emacs/libs/gnutls-3.1.8-w32/include
-IC:/emacs/libs/libiconv-1.14-2-mingw32-dev/include'
> So, somehow, compiling tramp-sh.el on Unix replaces the call to
> tramp-drop-volume-letter with nothing. Perhaps the condition, viz.
Yes, that is intended. tramp-drop-volume-letter is not needed on GNU/Linux.
> (defalias 'tramp-drop-volume-letter
> (if (memq system-type '(cygwin windows-nt)) <<<<<<<<<<<<<<<<<<<
> (lambda (name)
> "Cut off unnecessary drive letter from file NAME.
> The functions `tramp-*-handle-expand-file-name' call `expand-file-name'
> locally on a remote file name. When the local system is a W32 system
> but the remote system is Unix, this introduces a superfluous drive
> letter into the file name. This function removes it."
> (save-match-data
> (if (string-match "\\`[a-zA-Z]:/" name)
> (replace-match "/" nil t name)
> name)))
>
> 'identity))
>
> is expanded at byte-compile time, instead of at run time?
Looks like this, yes.
> And I guess whatever has this effect was introduced between the
> release of 24.2 and 24.3 (which was the trunk until 24.2 release).
Yes, it was changed in bzr commit #105625.
Does this mean, this declaration requests Tramp to be compiled on MS
Windows? I have used the official distribution from
<http://ftp.gnu.org/gnu/emacs/windows/>. It reports
In GNU Emacs 24.3.1 (i386-mingw-nt6.1.7600)
of 2013-03-17 on MARVIN
Windowing system distributor `Microsoft Corp.', version 6.1.7600
Configured using:
`configure --with-gcc (4.7) --cflags
-ID:/devel/emacs/libs/libXpm-3.5.8/include
-ID:/devel/emacs/libs/libXpm-3.5.8/src
-ID:/devel/emacs/libs/libpng-dev_1.4.3-1/include
-ID:/devel/emacs/libs/zlib-dev_1.2.5-2/include
-ID:/devel/emacs/libs/giflib-4.1.4-1/include
-ID:/devel/emacs/libs/jpeg-6b-4/include
-ID:/devel/emacs/libs/tiff-3.8.2-1/include
-ID:/devel/emacs/libs/gnutls-3.0.9/include
-ID:/devel/emacs/libs/libiconv-1.13.1-1-dev/include
-ID:/devel/emacs/libs/libxml2-2.7.8/include/libxml2'
We shall repackage Emacs 24.3 for MS Windows then. Other packages might
suffer from the same problem.
Best regards, Michael.
next prev parent reply other threads:[~2013-03-24 17:21 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-22 15:17 bug#14030: 24.3; expand-file-name expands remote file name on Windows incorrectly Vida Gábor
2013-03-22 16:02 ` Eli Zaretskii
2013-03-23 12:13 ` Michael Albinus
2013-03-23 20:23 ` Eli Zaretskii
2013-03-23 21:37 ` Michael Albinus
2013-03-24 16:10 ` Eli Zaretskii
2013-03-24 17:21 ` Michael Albinus [this message]
2013-03-24 17:35 ` Eli Zaretskii
2013-03-24 18:32 ` Michael Albinus
2013-03-24 19:23 ` Eli Zaretskii
2013-03-25 8:36 ` Michael Albinus
2013-03-25 9:01 ` Eli Zaretskii
2013-03-25 9:20 ` Michael Albinus
2013-03-25 9:55 ` Eli Zaretskii
2013-03-25 10:06 ` Michael Albinus
2013-03-27 0:11 ` Wolfgang Jenkner
2013-03-27 2:24 ` Stefan Monnier
2013-03-27 6:59 ` Eli Zaretskii
2013-03-26 0:35 ` Wolfgang Jenkner
2013-03-25 14:28 ` Stefan Monnier
2013-03-25 18:49 ` Stefan Monnier
2013-03-25 19:28 ` Michael Albinus
2013-03-25 20:32 ` Eli Zaretskii
2013-03-26 0:23 ` Stefan Monnier
2013-03-26 6:11 ` Eli Zaretskii
2013-03-24 16:11 ` Eli Zaretskii
2013-03-25 8:53 ` Vida Gábor
2020-09-09 13:56 ` Lars Ingebrigtsen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87620g67df.fsf@gmx.de \
--to=michael.albinus@gmx.de \
--cc=14030@debbugs.gnu.org \
--cc=eliz@gnu.org \
--cc=vidagabor@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).