[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el (read-fi
From: |
Stefan Monnier |
Subject: |
Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el (read-file-name-default): Bind `non-essential' to `t'. |
Date: |
Mon, 18 Jun 2012 16:35:13 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1.50 (gnu/linux) |
>>> Here's a recipe to reproduce the problem (revert the patch in
>>> minibuffer.el first):
>>> emacs -Q -l icomplete -f icomplete-mode
>>> C-x C-f /ssh:localhost:/ <TAB>
>> I don't see any problem here. Tramp requests a password, as expected
>> and as it should (since the user hit TAB, she clearly wants completion).
> As Drew said, it might disturb users. It wouldn't be necessary, if the
> default check wouldn't be `file-exsists-p', which provokes the password
> request. Something like
> (or (file-remote-p fn) (file-exists-p fn))
> should be better (again, I don't know where to change).
Why? The user wants to get the list of completions (that's why she hit
TAB), so she really wants to connect. Not connecting (as we currently
do) is a bug.
Stefan
- RE: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., (continued)
- RE: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Drew Adams, 2012/06/18
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Stefan Monnier, 2012/06/18
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Michael Albinus, 2012/06/19
- RE: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Drew Adams, 2012/06/19
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Michael Albinus, 2012/06/19
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Stefan Monnier, 2012/06/19
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Michael Albinus, 2012/06/19
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Michael Albinus, 2012/06/20
- RE: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el(read-file-name-default): Bind `non-essential' to `t'., Drew Adams, 2012/06/20
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el (read-file-name-default): Bind `non-essential' to `t'., Michael Albinus, 2012/06/18
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el (read-file-name-default): Bind `non-essential' to `t'.,
Stefan Monnier <=
- Re: [Emacs-diffs] /srv/bzr/emacs/trunk r108646: * minibuffer.el (read-file-name-default): Bind `non-essential' to `t'., Michael Albinus, 2012/06/18