On 12/31/2017 01:00 PM, 積丹尼 Dan Jacobson wrote:
$ man find

        -readable
               Matches files which are readable.  This takes into account 
access  control  lists  and  other  permissions
               artefacts  which  the -perm test ignores.  This test makes use 
of the access(2) system call, and so can be
               fooled by NFS servers which do UID mapping (or root-squashing), 
since many systems implement access(2)  in
               the client's kernel and so cannot make use of the UID mapping 
information held on the server.

Mention readable by whom, so one need not check the Info manual.

Thanks, the attached fixes it - I'll push next year.

Have a nice day,
Berny

>From 4b266176fb6b93f837842cad47c6dddf542fbd68 Mon Sep 17 00:00:00 2001
From: Bernhard Voelker <m...@bernhard-voelker.de>
Date: Sun, 31 Dec 2017 13:56:17 +0100
Subject: [PATCH] find.1: document that access()-based options apply to the
 current user
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* find/find.1 (-executable): Mention to whom the matching files are
accessible: to the current user.
(-readable): Likewise.
(-writable): Likewise.

Reported by 積丹尼 Dan Jacobson <jida...@jidanni.org> in
https://lists.gnu.org/r/bug-findutils/2017-12/msg00017.html
---
 find/find.1 | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/find/find.1 b/find/find.1
index 8b1320c1..0646b931 100644
--- a/find/find.1
+++ b/find/find.1
@@ -608,8 +608,9 @@ File is empty and is either a regular file or a directory.
 
 .IP \-executable
 Matches files which are executable and directories which are
-searchable (in a file name resolution sense).  This takes into account
-access control lists and other permissions artefacts which the
+searchable (in a file name resolution sense) by the current user.
+This takes into account access control lists and other permissions
+artefacts which the
 .B \-perm
 test ignores.  This test makes use of the
 .BR access (2)
@@ -862,8 +863,8 @@ This is no longer supported (and has been deprecated since 2005).  Use
 instead.
 
 .IP \-readable
-Matches files which are readable.  This takes into account access
-control lists and other permissions artefacts which the
+Matches files which are readable by the current user.  This takes into
+account access control lists and other permissions artefacts which the
 .B \-perm
 test ignores.  This test makes use of the
 .BR access (2)
@@ -972,8 +973,8 @@ See \-path.  This alternative is less portable than
 .BR \-path .
 
 .IP "\-writable"
-Matches files which are writable.  This takes into account access
-control lists and other permissions artefacts which the
+Matches files which are writable by the current user.  This takes into
+account access control lists and other permissions artefacts which the
 .B \-perm
 test ignores.  This test makes use of the
 .BR access (2)
-- 
2.15.1

Reply via email to