[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button
From: |
Michael Albinus |
Subject: |
bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button |
Date: |
Thu, 08 May 2014 19:04:22 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) |
Glenn Morris <rgm@gnu.org> writes:
> Michael Albinus wrote:
>
>>> ! (default-directory
>>> ! (if (and (not (file-remote-p default-directory))
>>> ! (file-accessible-directory-p default-directory))
>>> default-directory
>>> (expand-file-name "~/"))))
>>
>> I haven't checked further, but wouldn't this break url-handlers? See
>>
>> (progn
>> (url-handler-mode 1)
>> (file-remote-p "http://user@host/"))
>
> I don't see how? Default-directory above is the one that browse-url uses
> for the external browser that it starts.
Maybe I'm overreacting, but when you touch default-directory, file
operations from url-handlers might not work anymore.
(let ((default-directory "http://debbugs.gnu.org/"))
(url-handler-mode 1)
(expand-file-name "17245"))
returns a valid url which could be passed to browse-url.
I have no idea, whether browse-url and the operations in url-handlers
will be called together, 'tho. Maybe never.
But I have seen too many libraries which massage Tramp internals in
unexpected ways. So my personal alarm bell is ringing.
Feel free to ignore it.
Best regards, Michael.
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, (continued)
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Glenn Morris, 2014/05/06
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/07
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Sam Steingold, 2014/05/07
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Glenn Morris, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Glenn Morris, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Sam Steingold, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Glenn Morris, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button,
Michael Albinus <=
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Glenn Morris, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/08
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/09
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Stefan Monnier, 2014/05/09
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/12
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Stefan Monnier, 2014/05/12
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/12
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Stefan Monnier, 2014/05/12
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/12
- bug#17425: 24.4.50; tramp-cleanup-this-connection on push-button, Michael Albinus, 2014/05/12