From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Michael Albinus Newsgroups: gmane.emacs.devel 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 18:47:15 +0200 Message-ID: <87obog8t4c.fsf@gmx.de> References: <87k3z5ouz1.fsf@gmx.de> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1340038048 11672 80.91.229.3 (18 Jun 2012 16:47:28 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Mon, 18 Jun 2012 16:47:28 +0000 (UTC) Cc: emacs-devel@gnu.org To: Stefan Monnier Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Mon Jun 18 18:47:27 2012 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Sgf6s-0004wW-OO for ged-emacs-devel@m.gmane.org; Mon, 18 Jun 2012 18:47:26 +0200 Original-Received: from localhost ([::1]:59856 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Sgf6s-0004AV-Hl for ged-emacs-devel@m.gmane.org; Mon, 18 Jun 2012 12:47:26 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:44917) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Sgf6q-0004AP-8t for emacs-devel@gnu.org; Mon, 18 Jun 2012 12:47:25 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Sgf6o-0006uB-Em for emacs-devel@gnu.org; Mon, 18 Jun 2012 12:47:23 -0400 Original-Received: from mailout-de.gmx.net ([213.165.64.23]:53236) by eggs.gnu.org with smtp (Exim 4.71) (envelope-from ) id 1Sgf6o-0006tv-4Q for emacs-devel@gnu.org; Mon, 18 Jun 2012 12:47:22 -0400 Original-Received: (qmail invoked by alias); 18 Jun 2012 16:47:19 -0000 Original-Received: from p57BB98EC.dip0.t-ipconnect.de (EHLO detlef.gmx.de) [87.187.152.236] by mail.gmx.net (mp033) with SMTP; 18 Jun 2012 18:47:19 +0200 X-Authenticated: #3708877 X-Provags-ID: V01U2FsdGVkX1/tf8xcos3tX/xbtN3yoY7pGx/9v5TJXtRAUFNcLl vnZ+5pL8Uo/KE4 In-Reply-To: (Stefan Monnier's message of "Mon, 18 Jun 2012 10:03:05 -0400") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.1.50 (gnu/linux) X-Y-GMX-Trusted: 0 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 213.165.64.23 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:150989 Archived-At: Stefan Monnier writes: >> 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:/ > > 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). Or we spend a customer option for this, which will be the bound value of `non-essential' in `read-file-name-default'. > Stefan Best regards, Michael.