GNU bug report logs - #11000
24.0.94; Caching '--dired' is a bad idea

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 11000 in the body.
You can then email your comments to 11000 AT debbugs.gnu.org in the normal way.

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

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

From: Leo <sdl.web <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.0.94; Caching '--dired' is a bad idea
Date: Mon, 12 Mar 2012 19:45:52 +0800

If I change insert-directory-program from one supports --dired to one
not, I can no longer open any directory in dired-mode. For example on
OSX, I have the `ls' from BSD and another one from GNU coreutils
installed as `gls', not to mention opening remote directories.

I think caching --dired switch in dired-use-ls-dired is a bad idea.

Leo

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

From: Glenn Morris <rgm <at> gnu.org>
To: Leo <sdl.web <at> gmail.com>
Cc: 11000 <at> debbugs.gnu.org
Subject: Re: bug#11000: 24.0.94; Caching '--dired' is a bad idea
Date: Mon, 12 Mar 2012 12:47:31 -0400

Leo wrote:

> If I change insert-directory-program from one supports --dired to one
> not, I can no longer open any directory in dired-mode. For example on
> OSX, I have the `ls' from BSD and another one from GNU coreutils
> installed as `gls', not to mention opening remote directories.
>
> I think caching --dired switch in dired-use-ls-dired is a bad idea.

Not caching it is a worse idea.
There should simple be a note in the doc of insert-directory-program,
saying that if you change this, you may need to change
dired-use-ls-dired. If insert-directory-program were a defcustom
(currently, it is not even a user option), it could have a :set function
that makes the necessary change for you.

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

From: Leo <sdl.web <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#11000: 24.0.94; Caching '--dired' is a bad idea
Date: Tue, 13 Mar 2012 18:47:01 +0800

On 2012-03-13 00:47 +0800, Glenn Morris wrote:
> Not caching it is a worse idea.

Could you elaborate on this point? I really would like to know.

> There should simple be a note in the doc of insert-directory-program,
> saying that if you change this, you may need to change
> dired-use-ls-dired. If insert-directory-program were a defcustom
> (currently, it is not even a user option), it could have a :set
> function that makes the necessary change for you.

I agree.

Leo

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

From: Glenn Morris <rgm <at> gnu.org>
To: Leo <sdl.web <at> gmail.com>
Cc: 11000 <at> debbugs.gnu.org
Subject: Re: bug#11000: 24.0.94; Caching '--dired' is a bad idea
Date: Tue, 13 Mar 2012 19:47:42 -0400

Leo wrote:

> On 2012-03-13 00:47 +0800, Glenn Morris wrote:
>> Not caching it is a worse idea.
>
> Could you elaborate on this point? I really would like to know.

Just that it sounds very inefficient for everybody to have test what
options `ls' supports on every invocation, just because you are doing
something that sounds frankly rather odd.

There are many places in Emacs that test for features in external
commands the first time only, then make the IMO reasonable assumption
that the command isn't going to change underneath.
rmail-movemail-variant-in-use, find-exec-terminator, etc.

Message #21 received at 11000-done <at> debbugs.gnu.org (full text, mbox):

From: Leo Liu <sdl.web <at> gmail.com>
To: 11000-done <at> debbugs.gnu.org
Subject: Re: bug#11000: 24.0.94; Caching '--dired' is a bad idea
Date: Sun, 24 Mar 2013 20:01:30 +0800

Not a bug.

Previous Next

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