unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Yuri Khan <yuri.v.khan@gmail.com>
Cc: rpluim@gmail.com, gautier@gautierponsinet.xyz, emacs-devel@gnu.org
Subject: Re: Dired subdirectories & the ls option time-style with the %R sequence
Date: Wed, 24 May 2023 19:58:32 +0300	[thread overview]
Message-ID: <83wn0xmzt3.fsf@gnu.org> (raw)
In-Reply-To: <CAP_d_8VX4c2PRvAGKO54d-k5bdL2R698F9c5h103Vz435+oy8Q@mail.gmail.com> (message from Yuri Khan on Wed, 24 May 2023 23:41:37 +0700)

> From: Yuri Khan <yuri.v.khan@gmail.com>
> Date: Wed, 24 May 2023 23:41:37 +0700
> Cc: Gautier Ponsinet <gautier@gautierponsinet.xyz>, emacs-devel@gnu.org
> 
> On Wed, 24 May 2023 at 18:36, Robert Pluim <rpluim@gmail.com> wrote:
> >
> >     Gautier> Maybe `dired-insert-subdir' could search for "R" but exclude "%R"?
> >
> > Something like the following (untested) patch?
> >
> > -    (if (setq switches-have-R (and switches (string-match-p "R" switches)))
> > +    (if (setq switches-have-R (and switches (string-match-p "[^%]R" switches)))
> 
> This looks like a kludge. Will any occurrences of ‘R’ also be
> discarded in values for options such as ‘--hide’, ‘-I’|‘--ignore’, and
> any new options introduced later that take an arbitrary user-supplied
> string?
> 
> Also, will a ‘--recursive’ affect ‘dired-insert-subdir’ in the same
> way that removing ‘R’ is designed to prevent?

We sometimes install kludgey solutions if no better ideas are
available.  Do you have better ideas for fixing this?

Thanks.



  reply	other threads:[~2023-05-24 16:58 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-24 11:07 Dired subdirectories & the ls option time-style with the %R sequence Gautier Ponsinet
2023-05-24 11:36 ` Robert Pluim
2023-05-24 15:03   ` gautier
2023-05-25  6:45     ` Robert Pluim
2023-05-25 14:31       ` Gautier Ponsinet
2023-05-25 16:21         ` Robert Pluim
2023-05-27 10:33           ` Gautier Ponsinet
2023-05-27 11:07           ` Gautier Ponsinet
     [not found]           ` <draft-87wn0ukp7h.fsf@gautierponsinet.xyz>
2023-07-23  9:46             ` Gautier Ponsinet
2023-05-24 16:41   ` Yuri Khan
2023-05-24 16:58     ` Eli Zaretskii [this message]
2023-05-24 17:21       ` Yuri Khan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83wn0xmzt3.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=gautier@gautierponsinet.xyz \
    --cc=rpluim@gmail.com \
    --cc=yuri.v.khan@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).