Re: [bug #58941] wishlist: Support for -xattr -xattrname

2020-08-13 Thread Morgan Weetman
Hi Berny, On a side note.. an issue has been raised against my repo regarding compatibility with BSD-based find in MacOS and Solaris, apparently -xattr in those distributions doesn't take an argument. Could you tell me what the strategy is for handling compatibility between GNU find and BSD find?

Re: [bug #58941] wishlist: Support for -xattr -xattrname

2020-08-13 Thread Morgan Weetman
Hey Berny, I may have a week free in the middle of September, but I'll probably need assistance to knock it out in a week. Personally, I don't care about attribution, if anyone else wants to work on it to get it over the line you're welcome. thanks On Fri, 14 Aug 2020 at 05:32, Bernhard Voelker

[bug #57025] find -mtime will ignore large files

2020-08-13 Thread Bernhard Voelker
Follow-up Comment #8, bug #57025 (project findutils): 3rd: the texinfo manual already contained such a clarification, yet not in the place one would try to look for. The attached new patch fixes that (too). Have a nice day, Berny (file #49667)

[bug #58654] -perm -+w does not behave like chmod (POSIX)

2020-08-13 Thread Bernhard Voelker
Update of bug #58654 (project findutils): Category:None => documentation Status:None => Code Review ___ Follow-up Comment #5: The attached docu

Re: Add examples to show depth

2020-08-13 Thread Bernhard Voelker
On 2020-07-22 14:00, 積丹尼 Dan Jacobson wrote: > (info "(find) Directories") says > > -- Option: -maxdepth levels > Descend at most LEVELS (a non-negative integer) levels of > directories below the command line arguments. '-maxdepth 0' means > only apply the tests and actions to the

[bug #58205] Double dash option (--) mentioned only in manpage, unclear/nonworking

2020-08-13 Thread Bernhard Voelker
Update of bug #58205 (project findutils): Status: In Progress => Code Review ___ Follow-up Comment #3: Thanks for your comments. I suggest the attached documentation change. Have a nice day, Berny

[PATCH 6/6] doc: display version of the GNU findutils on the title page

2020-08-13 Thread Bernhard Voelker
The first page of the Texinfo documentation should display the package version the documentation corresponds to. Also change the title from "Finding Files" to "GNU Findutils ..." to better reflect the package name. These changes align the documentation better with that of other GNU packages like

[PATCH 5/6] maint: explicitly copy fdl-1.3 from gnulib

2020-08-13 Thread Bernhard Voelker
* Makefile.am (update-gnulib-to-latest): Prompted by a probably accidental difference between gnulib's fdl.texi and fdl-1.3.texi (to be fixed in GNU standards first), change the command to explicitly copy version 1.3 of the FDL. * doc/fdl.texi: Update from 'gnulib/doc/fdl-1.3.texi'. See https://li

[PATCH 2/6] doc: modernize example list of filesystem types for -fstype

2020-08-13 Thread Bernhard Voelker
* doc/find.texi (-fstype): Mention more common types. --- doc/find.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/find.texi b/doc/find.texi index 7a892dfe..1cb65066 100644 --- a/doc/find.texi +++ b/doc/find.texi @@ -1512,7 +1512,7 @@ filesystem types vary among differ

[PATCH 4/6] doc: change "allows to " to "allows ing"

2020-08-13 Thread Bernhard Voelker
* doc/find.texi: Use gerund after "allows". As a side effect, this also avoids passive voice in some cases; active voice is usually clearer. --- doc/find.texi | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/doc/find.texi b/doc/find.texi index 1cb650

[PATCH 3/6] maint: fix typo in comment

2020-08-13 Thread Bernhard Voelker
* find/ftsfind.c (inside_dir): s/currentl/currently/ --- find/ftsfind.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/find/ftsfind.c b/find/ftsfind.c index b4bccb4d..783148c5 100644 --- a/find/ftsfind.c +++ b/find/ftsfind.c @@ -126,7 +126,7 @@ inside_dir (int dir_fd) else

[PATCH 0/6] several doc improvements and fixes

2020-08-13 Thread Bernhard Voelker
[PATCH 1/6] doc: remove fts from configuration chapter [PATCH 2/6] doc: modernize example list of filesystem types for [PATCH 3/6] maint: fix typo in comment [PATCH 4/6] doc: change "allows to " to "allows ing" [PATCH 5/6] maint: explicitly copy fdl-1.3 from gnulib [PATCH 6/6] doc: display version

[PATCH 1/6] doc: remove fts from configuration chapter

2020-08-13 Thread Bernhard Voelker
The corresponding configure option was removed with commit 42a28b0dfb. * doc/find.texi (node fts): Remove section. --- doc/find.texi | 16 1 file changed, 16 deletions(-) diff --git a/doc/find.texi b/doc/find.texi index aeb59cd3..7a892dfe 100644 --- a/doc/find.texi +++ b/doc/fin

Re: [bug #58941] wishlist: Support for -xattr -xattrname

2020-08-13 Thread Bernhard Voelker
On 2020-08-13 04:37, Morgan Weetman wrote: > Done.. I just made the repo public: https://gitlab.com/mweetman/findutils Thanks for sharing. I was just about to ask about it. The patch from that time was already ~80% complete. Do you have time and are you willing to make it round in the next ~2-3 m

Re: [bug #58941] wishlist: Support for -xattr -xattrname

2020-08-13 Thread Kevin Locke
Hi Morgan, On Thu, 2020-08-13 at 12:02 +1000, Morgan Weetman wrote: > On Thu, 13 Aug 2020 at 00:34, Kevin Locke wrote: >> It would be great if find could support searching for files based on the >> presence/absence of extended attributes (xattrs). > > I wrote support for this a few years ago but

Re: [bug #58941] wishlist: Support for -xattr -xattrname

2020-08-13 Thread Kevin Locke
On Thu, 2020-08-13 at 12:37 +1000, Morgan Weetman wrote: > On Thu, 13 Aug 2020 at 12:21, Kevin Locke wrote: >> On Thu, 2020-08-13 at 12:02 +1000, Morgan Weetman wrote: >>> On Thu, 13 Aug 2020 at 00:34, Kevin Locke wrote: It would be great if find could support searching for files based on the