GNU bug report logs - #10489
24.0.92; dired-do-copy may create infinite directory hierarchy

Previous Next

Package: emacs;

Reported by: michael_heerdegen <at> web.de

Date: Thu, 12 Jan 2012 19:36:01 UTC

Severity: important

Tags: patch

Merged with 11130

Found in version 24.0.92

Done: Chong Yidong <cyd <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #59 received at 10489 <at> debbugs.gnu.org (full text, mbox):

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Juanma Barranquero <lekktu <at> gmail.com>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 10489 <at> debbugs.gnu.org,
	Thierry Volpiatto <thierry.volpiatto <at> gmail.com>
Subject: Re: bug#10489: 24.0.92;
	dired-do-copy may create infinite directory hierarchy
Date: Fri, 13 Jan 2012 14:32:32 +0100
Juanma Barranquero <lekktu <at> gmail.com> writes:

>>  (compare-strings (file-truename (expand-file-name name1 dir)) 0 nil
>>                   (file-truename (expand-file-name name2 dir)) 0 nil
>>                   read-file-name-completion-ignore-case))
>
> I think we should push for it to be added to Emacs, if only for the
> thrill of having a ChangeLog entry
>
> 2012-01-13  Michael Albinus <michael.albinus <at> gmx.de>
> 	    Juanma Barranquero  <lekktu <at> gmail.com>
> 	    Eli Zaretskii <eliz <at> gnu.org>
>
> 	* files.el (file-name-compare): New function.
>
> i.e., three authors for a three-line function (plus header and comments) ;-)

Perfect reasoning, I'm for it :-)

Thinking about, `file-truename' would need some massage for the remote
case. Currently, it just expands symlinks. But it could also do some
normalization for the user and host name. For example:

(file-truename "/ssh:127.0.0.1:/path/to/symlink")
 => "/ssh:albinus <at> localhost:/extend/path/to/file"

But that's for Emacs 24.2, of course.

> Juanma

Best regards, Michael.




This bug report was last modified 13 years and 58 days ago.

Previous Next


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