[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Proposal: ls-lisp.el handle --group-directories-first flag
From: |
Arthur Miller |
Subject: |
Re: Proposal: ls-lisp.el handle --group-directories-first flag |
Date: |
Sat, 24 Jul 2021 16:58:08 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) |
Eli Zaretskii <eliz@gnu.org> writes:
>> From: Arthur Miller <arthur.miller@live.com>
>> Cc: emacs-devel@gnu.org
>> Date: Sat, 24 Jul 2021 13:54:49 +0200
>>
>> > Btw, in my testing, with ls-lisp-dirs-first bound non-nil, the
>> > directories are shown in their correct alphabetically sorted order.
>> > So I'm unsure what problems you saw and in which scenario. Can you
>> > tell more about this issue?
When I was testing, if ls-lisp-dirs-first is nil, and
--group-directories-first is specified, dirs come out in reverse order
in new buffer. Just reverting the buffer didn't show this. I don't know
why it is so. I just tested with emacs built last night from the
master (though only on Arch Linux). Either kill the dired buffer and
open again, or just navigate to another dir.
>> Ok, I'll test more.
>
> Thanks.
I remember now what was going on: when --group-directories-first is
specified, the -U option does not have effect according to gnu ls
manual. Anyway, since we let-bind the ls-lisp-dirs-first, I can just set
that one to 't and get same effect.
>> Have you seen the other patch, where flags are sanitized, that would
>> make ls-lisp accept all the flags as gnu ls, but ignore those it does
>> not undrstand.
>
> Yes, it looks OK to me.
If you are OK with the patch2, then here is one without -U on top of
that one.
ls-lisp.patch
Description: Text Data
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, (continued)
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/16
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Arthur Miller, 2021/07/16
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/16
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Arthur Miller, 2021/07/16
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/17
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Arthur Miller, 2021/07/19
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/24
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/24
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Arthur Miller, 2021/07/24
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/24
- Re: Proposal: ls-lisp.el handle --group-directories-first flag,
Arthur Miller <=
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/24
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Arthur Miller, 2021/07/24
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/25
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Arthur Miller, 2021/07/25
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Eli Zaretskii, 2021/07/25
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Arthur Miller, 2021/07/25
- Re: Proposal: ls-lisp.el handle --group-directories-first flag, Michael Albinus, 2021/07/25