GNU bug report logs -
#29513
sorting find-dired
Previous Next
Reported by: "Roland Winkler" <winkler <at> gnu.org>
Date: Fri, 1 Dec 2017 00:51:01 UTC
Severity: wishlist
Merged with 36110,
36123
Found in version 25.2
Fixed in version 27.1
Done: "Roland Winkler" <winkler <at> gnu.org>
Bug is archived. No further changes may be made.
Full log
Message #20 received at 29513 <at> debbugs.gnu.org (full text, mbox):
On Fri Dec 1 2017 Drew Adams wrote:
> > If we do this - wouldn't supporting other sort predicates also make
> > sense (e.g. sort by last change time)?
>
> My thought exactly. Pretty much anywhere we sort stuff
> it makes sense to ask ourselves whether it might be
> good to let users specify the sort predicate/order.
Fine with me, I'd be happy to be able to use such a feature.
The only question is how to achieve that. The point is that (as far
as I understand these things) something like dired-sort-toggle-or-edit
ultimately relies on the shell command ls to sort its output.
Sorting the output of find by predicates such as last change time
requires to parse its output more carefully. I believe that
currently dired's parsing capabilities are restricted to identifying
the file name, and my patch for find-dired-sentinel relies on that.
In other words, other predicates require somewhat more work than my
patch.
(All this reminds me of when I wrote proced.el years ago. Initially
this code relied on the shell command ps that comes with zillions of
flavors on different OSs, which made the code rather fragile.
Things got much cleaner when Eli wrote the built-ins
list-system-processes and process-attributes (thank you Eli!!). But
I expect that we do not want to re-invent the shell command find for
Emacs. -- Phrased differently: I do not know how many flavors of
find we may have to cover to sort its output by predicates such as
last change time.)
This bug report was last modified 5 years and 343 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.