[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames
From: |
Eli Zaretskii |
Subject: |
bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames |
Date: |
Mon, 04 Sep 2023 15:17:07 +0300 |
> Cc: 64791@debbugs.gnu.org, manuel@ledu-giraud.fr
> Date: Sun, 03 Sep 2023 22:25:30 +0300
> From: Eli Zaretskii <eliz@gnu.org>
>
> > -(defvar insert-directory-program (purecopy "ls")
> > +(defvar insert-directory-program
> > + (if (and (memq system-type '(berkeley-unix darwin))
> > + (executable-find "gls"))
> > + (purecopy "gls")
> > + (purecopy "ls"))
> > "Absolute or relative name of the `ls'-like program.
> > This is used by `insert-directory' and `dired-insert-directory'
> > \(thus, also by `dired').")
>
> If we believe "gls" is the only popular name, I guess that's good
> enough.
Actually, I take this back: this defvar will be computed when files.el
is loaded, which is at dump time. So this is not TRT, since the
conditions and the environment at dump time could be very different
from those at run time. It could be even on another machine.
So I once again suggest to make this a defcustom. This has two
advantages:
. it will be evaluated at Emacs startup, not at dump time
. it will allow users to customize the value and thus avoid costly
search for the program and prevent finding the wrong version of
'ls'
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, (continued)
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Eli Zaretskii, 2023/09/03
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/03
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Eli Zaretskii, 2023/09/03
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/04
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Manuel Giraud, 2023/09/04
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Po Lu, 2023/09/04
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Manuel Giraud, 2023/09/04
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/04
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Po Lu, 2023/09/04
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/05
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames,
Eli Zaretskii <=
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/05
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/05
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Eli Zaretskii, 2023/09/06
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/24
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Eli Zaretskii, 2023/09/24
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/25
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Manuel Giraud, 2023/09/24
- bug#64791: 30.0.50; [PATCH] Fix dired mismatch on some filenames, Stefan Kangas, 2023/09/24