GNU bug report logs - #508
emacs 22.2.1 becomes confused

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 508 in the body.
You can then email your comments to 508 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: "Bruce Korb" <bruce.korb <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Cc: "Bruce Korb" <bkorb <at> gnu.org>
Subject: emacs 22.2.1 becomes confused
Date: Mon, 30 Jun 2008 08:03:12 -0700

Hi,

I recently re-installed my system.  For various reasons not worth
going into here,
I renamed my home directory from /home/foo to /home/Foo and re-created my
"foo" user name.  It is not possible to edit anything in the /home/Foo hierarchy
with emacs.  At all.  I can cd to the correct directory and invoke "emacs file"
and it will try to open the equivalently named file under /home/foo.  Somebody
somewhere fiddled some code such that tab-completion works correctly, but
when it comes to actually "visiting" the file, the blasted thing substitutes
"$HOME" for "/home/Foo".  Someone please be kind enough to find that
"strncasecmp" and replace it with "strncmp" and put out an emergency fix.

Thank you.  Regards, Bruce

$ /usr/bin/emacs --version
GNU Emacs 22.2.1
Copyright (C) 2008 Free Software Foundation, Inc.

Message #10 received at 508 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Sven Joachim <svenjoac <at> gmx.de>
To: Bruce Korb <bruce.korb <at> gmail.com>
Cc: 508 <at> debbugs.gnu.org
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Mon, 30 Jun 2008 19:53:51 +0200

On 2008-06-30 17:03 +0200, Bruce Korb wrote:

> I recently re-installed my system.  For various reasons not worth
> going into here,
> I renamed my home directory from /home/foo to /home/Foo and re-created my
> "foo" user name.  It is not possible to edit anything in the /home/Foo hierarchy
> with emacs.  At all.  I can cd to the correct directory and invoke "emacs file"
> and it will try to open the equivalently named file under /home/foo.  Somebody
> somewhere fiddled some code such that tab-completion works correctly, but
> when it comes to actually "visiting" the file, the blasted thing substitutes
> "$HOME" for "/home/Foo".

This is with HOME=/home/foo, I suppose?  Because when I tried to
reproduce it, I set $HOME to /home/Foo and files beneath /home/foo
became inaccessible.

>  Someone please be kind enough to find that
> "strncasecmp" and replace it with "strncmp" and put out an emergency fix.

Actually, this is not a problem in the C code, but rather in Lisp, in
the function abbreviate-file-name.

The following patch tries to take care of this by matching the home
directory case-sensitively on systems which usually have such
filesystems, could you please test it?

--8<---------------cut here---------------start------------->8---
--- files.el.~1.985.~	2008-06-11 07:54:21.000000000 +0200
+++ files.el	2008-06-30 19:37:37.000000000 +0200
@@ -1412,7 +1412,10 @@
 	     (file-exists-p (file-name-directory
 			     (substring filename (1- (match-end 0))))))
 	(setq filename (substring filename (1- (match-end 0)))))
-    (let ((tail directory-abbrev-alist))
+    (let ((tail directory-abbrev-alist)
+	  (case-fold-search
+	   (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))
+	       t nil)))
       ;; If any elt of directory-abbrev-alist matches this name,
       ;; abbreviate accordingly.
       (while tail
--8<---------------cut here---------------end--------------->8---

Changelog entry:

2008-06-30  Sven Joachim  <svenjoac <at> gmx.de>

	* files.el (abbreviate-file-name): Match the home directory
	case-sensitively on systems that have case-sensitive filesystems.

Regards,
        Sven

Message #15 received at 508 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: "Bruce Korb" <bruce.korb <at> gmail.com>
To: "Sven Joachim" <svenjoac <at> gmx.de>
Cc: 508 <at> debbugs.gnu.org
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Mon, 30 Jun 2008 11:12:57 -0700

On Mon, Jun 30, 2008 at 10:53 AM, Sven Joachim <svenjoac <at> gmx.de> wrote:
> The following patch tries to take care of this by matching the home
> directory case-sensitively on systems which usually have such
> filesystems, could you please test it?

Sure, but given you reproduced the problem and that this fixed it
for you, I'm guessing it will work.  Now, where does this file live?
I know I can find it if I look hard enough, but the easy search
only yielded /usr/share/emacs/22.2/lisp/files.elc
Thanks - Bruce

> --8<---------------cut here---------------start------------->8---
> --- files.el.~1.985.~   2008-06-11 07:54:21.000000000 +0200
> +++ files.el    2008-06-30 19:37:37.000000000 +0200
> @@ -1412,7 +1412,10 @@
>             (file-exists-p (file-name-directory
>                             (substring filename (1- (match-end 0))))))
>        (setq filename (substring filename (1- (match-end 0)))))
> -    (let ((tail directory-abbrev-alist))
> +    (let ((tail directory-abbrev-alist)
> +         (case-fold-search
> +          (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))
> +              t nil)))
>       ;; If any elt of directory-abbrev-alist matches this name,
>       ;; abbreviate accordingly.
>       (while tail
> --8<---------------cut here---------------end--------------->8---

Message #20 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Sven Joachim <svenjoac <at> gmx.de>, 508 <at> debbugs.gnu.org
Cc: bruce.korb <at> gmail.com, bug-gnu-emacs <at> gnu.org
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Mon, 30 Jun 2008 22:57:21 +0300

> From: Sven Joachim <svenjoac <at> gmx.de>
> Date: Mon, 30 Jun 2008 19:53:51 +0200
> Cc: 508 <at> emacsbugs.donarmstrong.com
> 
> +	  (case-fold-search
> +	   (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))

You forgot cygwin in this list.

Message #30 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Bruce Korb <bruce.korb <at> gmail.com>, 508 <at> debbugs.gnu.org
Cc: svenjoac <at> gmx.de, bug-gnu-emacs <at> gnu.org
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Mon, 30 Jun 2008 23:01:37 +0300

> Date: Mon, 30 Jun 2008 11:12:57 -0700
> From: "Bruce Korb" <bruce.korb <at> gmail.com>
> Cc: 508 <at> emacsbugs.donarmstrong.com
> 
> On Mon, Jun 30, 2008 at 10:53 AM, Sven Joachim <svenjoac <at> gmx.de> wrote:
> > The following patch tries to take care of this by matching the home
> > directory case-sensitively on systems which usually have such
> > filesystems, could you please test it?
> 
> Sure, but given you reproduced the problem and that this fixed it
> for you, I'm guessing it will work.  Now, where does this file live?
> I know I can find it if I look hard enough, but the easy search
> only yielded /usr/share/emacs/22.2/lisp/files.elc

You probably didn't installed the Lisp sources.  But don't hurry to do
that yet: files.el is preloaded into Emacs, so to make that change you
will have to rebuild Emacs, not just patch files.el.

Message #40 received at 508 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Sven Joachim <svenjoac <at> gmx.de>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 508 <at> debbugs.gnu.org, bruce.korb <at> gmail.com
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Mon, 30 Jun 2008 22:14:02 +0200

On 2008-06-30 21:57 +0200, Eli Zaretskii wrote:

>> From: Sven Joachim <svenjoac <at> gmx.de>
>> Date: Mon, 30 Jun 2008 19:53:51 +0200
>> Cc: 508 <at> emacsbugs.donarmstrong.com
>> 
>> +	  (case-fold-search
>> +	   (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))
>
> You forgot cygwin in this list.

I just pasted the list from the read-file-name-completion-ignore-case
defcustom in minibuffer.el.  Should that list include cygwin as well?

Sven

Message #45 received at 508 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: "Lennart Borgman (gmail)" <lennart.borgman <at> gmail.com>
To: Sven Joachim <svenjoac <at> gmx.de>, 508 <at> debbugs.gnu.org
Cc: Eli Zaretskii <eliz <at> gnu.org>, bruce.korb <at> gmail.com
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Mon, 30 Jun 2008 22:30:03 +0200

Sven Joachim wrote:
> On 2008-06-30 21:57 +0200, Eli Zaretskii wrote:
> 
>>> From: Sven Joachim <svenjoac <at> gmx.de>
>>> Date: Mon, 30 Jun 2008 19:53:51 +0200
>>> Cc: 508 <at> emacsbugs.donarmstrong.com
>>>
>>> +	  (case-fold-search
>>> +	   (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))
>> You forgot cygwin in this list.
> 
> I just pasted the list from the read-file-name-completion-ignore-case
> defcustom in minibuffer.el.  Should that list include cygwin as well?


Shouldn't these things be defsubt so that it easier to change and maintain?

Message #50 received at 508 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Sven Joachim <svenjoac <at> gmx.de>
Cc: 508 <at> debbugs.gnu.org, bruce.korb <at> gmail.com
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Tue, 01 Jul 2008 05:58:48 +0300

> From: Sven Joachim <svenjoac <at> gmx.de>
> Cc: 508 <at> emacsbugs.donarmstrong.com,  bruce.korb <at> gmail.com
> Date: Mon, 30 Jun 2008 22:14:02 +0200
> 
> On 2008-06-30 21:57 +0200, Eli Zaretskii wrote:
> 
> >> From: Sven Joachim <svenjoac <at> gmx.de>
> >> Date: Mon, 30 Jun 2008 19:53:51 +0200
> >> Cc: 508 <at> emacsbugs.donarmstrong.com
> >> 
> >> +	  (case-fold-search
> >> +	   (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))
> >
> > You forgot cygwin in this list.
> 
> I just pasted the list from the read-file-name-completion-ignore-case
> defcustom in minibuffer.el.  Should that list include cygwin as well?

Yes, I think so.

Message #55 received at 508 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: "Lennart Borgman (gmail)" <lennart.borgman <at> gmail.com>
Cc: svenjoac <at> gmx.de, 508 <at> debbugs.gnu.org, bruce.korb <at> gmail.com
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Tue, 01 Jul 2008 06:00:27 +0300

> Date: Mon, 30 Jun 2008 22:30:03 +0200
> From: "Lennart Borgman (gmail)" <lennart.borgman <at> gmail.com>
> CC: Eli Zaretskii <eliz <at> gnu.org>, bruce.korb <at> gmail.com
> 
> Sven Joachim wrote:
> > On 2008-06-30 21:57 +0200, Eli Zaretskii wrote:
> > 
> >>> From: Sven Joachim <svenjoac <at> gmx.de>
> >>> Date: Mon, 30 Jun 2008 19:53:51 +0200
> >>> Cc: 508 <at> emacsbugs.donarmstrong.com
> >>>
> >>> +	  (case-fold-search
> >>> +	   (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))
> >> You forgot cygwin in this list.
> > 
> > I just pasted the list from the read-file-name-completion-ignore-case
> > defcustom in minibuffer.el.  Should that list include cygwin as well?
> 
> 
> Shouldn't these things be defsubt so that it easier to change and maintain?

I'm not sure I understand what you are suggesting.  Please explain.

Message #60 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Kevin Rodgers <kevin.d.rodgers <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Mon, 30 Jun 2008 21:56:33 -0600

Sven Joachim wrote:
> On 2008-06-30 21:57 +0200, Eli Zaretskii wrote:
> 
>>> From: Sven Joachim <svenjoac <at> gmx.de>
>>> Date: Mon, 30 Jun 2008 19:53:51 +0200
>>> Cc: 508 <at> emacsbugs.donarmstrong.com
>>>
>>> +	  (case-fold-search
>>> +	   (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))
>> You forgot cygwin in this list.
> 
> I just pasted the list from the read-file-name-completion-ignore-case
> defcustom in minibuffer.el.

Why not bind case-fold-search to the value of
read-file-name-completion-ignore-case there?

> Should that list include cygwin as well?

-- 
Kevin Rodgers
Denver, Colorado, USA

Message #65 received at 508 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Sven Joachim <svenjoac <at> gmx.de>
To: Kevin Rodgers <kevin.d.rodgers <at> gmail.com>
Cc: 508 <at> debbugs.gnu.org
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Tue, 01 Jul 2008 08:28:07 +0200

On 2008-07-01 05:56 +0200, Kevin Rodgers wrote:

> Sven Joachim wrote:
>> On 2008-06-30 21:57 +0200, Eli Zaretskii wrote:
>>
>>>> From: Sven Joachim <svenjoac <at> gmx.de>
>>>> Date: Mon, 30 Jun 2008 19:53:51 +0200
>>>> Cc: 508 <at> emacsbugs.donarmstrong.com
>>>>
>>>> +	  (case-fold-search
>>>> +	   (if (memq system-type '(ms-dos windows-nt darwin macos vax-vms axp-vms))
>>> You forgot cygwin in this list.
>>
>> I just pasted the list from the read-file-name-completion-ignore-case
>> defcustom in minibuffer.el.
>
> Why not bind case-fold-search to the value of
> read-file-name-completion-ignore-case there?

That would be wrong, IMO.  Users may or may not prefer filename
completion to be case sensitive regardless of their filesystems, but the
code in question really depends on the actual feature of the filesystem.

There could be a new defvar (or defconst), like this:

(defvar case-insensitive-systems-alist
  '(ms-dos windows-nt cygwin darwin macos vax-vms axp-vms)
  "List of systems who have case insensitive filesystems.")

which could then be used in code that depends on the filesystem being
case sensitive or not, and in the read-file-name-completion-ignore-case
defcustom as well.

Sven

Message #70 received at 508 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Sven Joachim <svenjoac <at> gmx.de>
Cc: 508 <at> debbugs.gnu.org, Kevin Rodgers <kevin.d.rodgers <at> gmail.com>
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Thu, 03 Jul 2008 00:54:57 -0400

> There could be a new defvar (or defconst), like this:

> (defvar case-insensitive-systems-alist
>   '(ms-dos windows-nt cygwin darwin macos vax-vms axp-vms)
>   "List of systems who have case insensitive filesystems.")

Actually, it should not be a variable but a function (and even
a file-name operation, obeying file-name-handlers):

   (directory-case-sensitive-p FILE)


-- Stefan

Message #75 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>, 508 <at> debbugs.gnu.org
Cc: svenjoac <at> gmx.de, kevin.d.rodgers <at> gmail.com, 508 <at> debbugs.gnu.org,
        bug-gnu-emacs <at> gnu.org
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Fri, 04 Jul 2008 13:04:22 +0300

> Resent-From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Resent-To: bug-submit-list <at> donarmstrong.com
> Resent-CC: Emacs Bugs <bug-gnu-emacs <at> gnu.org>
> Resent-Sender: don <at> donarmstrong.com
> From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Date: Thu, 03 Jul 2008 00:54:57 -0400
> Cc: Kevin Rodgers <kevin.d.rodgers <at> gmail.com>, 508 <at> emacsbugs.donarmstrong.com
> 
> > There could be a new defvar (or defconst), like this:
> 
> > (defvar case-insensitive-systems-alist
> >   '(ms-dos windows-nt cygwin darwin macos vax-vms axp-vms)
> >   "List of systems who have case insensitive filesystems.")
> 
> Actually, it should not be a variable but a function (and even
> a file-name operation, obeying file-name-handlers):
> 
>    (directory-case-sensitive-p FILE)

100% agreement: with the proliferation of network-mounted filesystems,
there's no longer a deterministic way to infer whether a filesystem is
case-sensitive or not from the OS brand.  Instead, we should query the
filesystem about that, and the function suggested by Stefan should do
that (when there're no file-name-handlers for the filesystem).

Message #85 received at 508-done <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Chong Yidong <cyd <at> stupidchicken.com>
To: 508-done <at> debbugs.gnu.org
Subject: Re: bug#508: emacs 22.2.1 becomes confused
Date: Thu, 31 Jul 2008 17:17:07 -0400

Since no one has offered to write a directory-case-sensitive-p function,
I've checked in Sven's first, simpler patch.  Even if it doesn't do the
completely right thing in the failure case (e.g., a vfat home directory
on GNU/Linux), all that does is fail to replace $HOME with ~, which is
hardly fatal.

After the release, if someone wants to write directory-case-sensitive-p,
we can switch to that.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.