all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Lennart Borgman <lennart.borgman@gmail.com>
To: Stefan Monnier <monnier@iro.umontreal.ca>,
	Michael Albinus <michael.albinus@gmx.de>
Cc: 5447@debbugs.gnu.org
Subject: bug#5447: closed by Eli Zaretskii <eliz@gnu.org> (Re: bug#5447: 23.1.91; load-file fails for C:/the-file.el.gz)
Date: Fri, 22 Jan 2010 19:03:42 +0100	[thread overview]
Message-ID: <e01d8a51001221003o39881a61r669796f6519b0b2c@mail.gmail.com> (raw)
In-Reply-To: <jwv636ukv0i.fsf-monnier+emacs@gnu.org>

On Fri, Jan 22, 2010 at 4:41 PM, Stefan Monnier
<monnier@iro.umontreal.ca> wrote:
>>> The interesting thing is that jka-compr-load is not even called if
>>> the file is not in the root directory of a drive.  I will try to
>>> look into that now.
>> This is actually trivial: a file that is not in C:/ is not considered
>> a remote file by Tramp handlers.  Case closed.
>> Like Lennart, I think that considering X:/ files on Windows to be
>> remote is a bad idea.  If it didn't have any serious consequences, I'd
>> just shrug.  But the recent bug reports indicate that it does have
>> very serious consequences, and is quite an annoyance.
>> Can we please get rid of that misfeature?
>
> I generally agree, but I'd like to see a concrete patch for it first.


Maybe the patch is not the problem. In the discussion around bug 5448
Michael wrote as an answer to my question:


On Wed, Jan 20, 2010 at 10:00 AM, Michael Albinus
<michael.albinus@gmx.de> wrote:
> Lennart Borgman <lennart.borgman@gmail.com> writes:
>
>>>> However I wonder why those files at all are interesting for tramp. I
>>>> know little about tramp, but does not remote file names always start
>>>> with something like "/ssh:", "/ftp:", "/telnet:" etc?
>>>>
>>>> If so why look for file names starting with "c:/"?
>>>
>>> Some packages ( I don't remember which ones) do add the volume letter to
>>> file names, which haven't one. Tramp silently removes the volume letter
>>> then, and continues.
>>
>> But is not that a bug in those packages then? Does not trying to fix
>> it in Tramp introduce this new bug?
>
> In theory, you are right. But Tramp supports also GNU Emacs 21, 22, and
> XEmacs. This makes it impossible to fix all involved packages.
>
> Best regards, Michael.


I do not understand exactly what Michael means with "some packages ...
do add the volume letter". However he says there is some problem
there. I still think this is a good time to fix it.

Looking at it again I get more confused. Does not the same thing
happen on un*x like file systems? I mean does not all that stuff Eli
mentioned get loaded if you try to complete a file name in the root on
those systems too? (But perhaps you do not bump into the problem there
that often?)

At least it looks to me from the pattern that this must happen there too:

   ("\\`\\([a-zA-Z]:\\)?/[^/]*\\'" . tramp-completion-file-name-handler)

Perhaps the right thing to do is to only match "/ssh:", "/ftp:",
"/telnet:" etc and only bring in Tramp if those things matches?






  reply	other threads:[~2010-01-22 18:03 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <83hbqe77l7.fsf@gnu.org>
2010-01-22  9:01 ` bug#5447: 23.1.91; load-file fails for C:/the-file.el.gz Eli Zaretskii
2010-01-22 10:34   ` bug#5447: marked as done (23.1.91; load-file fails for C:/the-file.el.gz) Emacs bug Tracking System
     [not found]   ` <handler.5447.D5447.126415640115246.notifdone@debbugs.gnu.org>
2010-01-22 10:56     ` bug#5447: closed by Eli Zaretskii <eliz@gnu.org> (Re: bug#5447: 23.1.91; " Eli Zaretskii
2010-01-22 15:41       ` Stefan Monnier
2010-01-22 18:03         ` Lennart Borgman [this message]
2010-01-22 21:30           ` Michael Albinus
2010-01-22 18:05         ` Eli Zaretskii
2010-01-22 21:37           ` Michael Albinus
2010-01-23  8:13             ` Eli Zaretskii

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e01d8a51001221003o39881a61r669796f6519b0b2c@mail.gmail.com \
    --to=lennart.borgman@gmail.com \
    --cc=5447@debbugs.gnu.org \
    --cc=michael.albinus@gmx.de \
    --cc=monnier@iro.umontreal.ca \
    /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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.