]> git.ipfire.org Git - thirdparty/coreutils.git/commitdiff
doc: fix stale --dereference info for chown/chgrp
authorPádraig Brady <P@draigBrady.com>
Sun, 17 Mar 2024 12:18:25 +0000 (12:18 +0000)
committerPádraig Brady <P@draigBrady.com>
Mon, 18 Mar 2024 18:40:21 +0000 (18:40 +0000)
Following v5.2.1-679-g7e29ef8b8 symlinks specified on the command line
no longer induce an error if lchown() is not supported on the system.

* doc/coreutils.texi (chown invocation, chgrp invocation): Adjust
accordingly, and also use a macro to avoid duplication.
* src/chown-core.c: Use our more standard is_ENOTSUP() wrapper
in the code related to this.

doc/coreutils.texi
src/chown-core.c

index d07ed7e76ab60b3e27618c7b409921b79fda0d76..09e3e9d3f509fc0de484936196a2620aaab2aba1 100644 (file)
@@ -11656,9 +11656,10 @@ though still not perfect:
 @end macro
 @chownFromOption{chown}
 
+@macro symlinkRefOpts
 @item --dereference
 @opindex --dereference
-@cindex symbolic links, changing owner
+@cindex symbolic links, changing owner, group
 @findex lchown
 Do not act on symbolic links themselves but rather on what they point to.
 This is the default when not operating recursively.
@@ -11673,10 +11674,9 @@ This is the default when not operating recursively.
 Act on symbolic links themselves instead of what they point to.
 This mode relies on the @code{lchown} system call.
 On systems that do not provide the @code{lchown} system call,
-@command{chown} fails when a file specified on the command line
-is a symbolic link.
-By default, no diagnostic is issued for symbolic links encountered
-during a recursive traversal, but see @option{--verbose}.
+no diagnostic is issued, but see @option{--verbose}.
+@end macro
+@symlinkRefOpts
 
 @item --preserve-root
 @opindex --preserve-root
@@ -11790,27 +11790,7 @@ changed.
 
 @chownFromOption{chgrp}
 
-@item --dereference
-@opindex --dereference
-@cindex symbolic links, changing owner
-@findex lchown
-Do not act on symbolic links themselves but rather on what they point to.
-This is the default when not operating recursively.
-@warnOptDerefWithRec
-
-@item -h
-@itemx --no-dereference
-@opindex -h
-@opindex --no-dereference
-@cindex symbolic links, changing group
-@findex lchown
-Act on symbolic links themselves instead of what they point to.
-This mode relies on the @code{lchown} system call.
-On systems that do not provide the @code{lchown} system call,
-@command{chgrp} fails when a file specified on the command line
-is a symbolic link.
-By default, no diagnostic is issued for symbolic links encountered
-during a recursive traversal, but see @option{--verbose}.
+@symlinkRefOpts
 
 @item --preserve-root
 @opindex --preserve-root
index fb5d4fe17706005f95db183badbf4d8ea5b529fa..5776c6a79aa28a1bc0cb6eb8d1d65af9a504cb4e 100644 (file)
@@ -420,7 +420,7 @@ change_file_owner (FTS *fts, FTSENT *ent,
           /* Ignore any error due to lack of support; POSIX requires
              this behavior for top-level symbolic links with -h, and
              implies that it's required for all symbolic links.  */
-          if (!ok && errno == EOPNOTSUPP)
+          if (!ok && is_ENOTSUP (errno))
             {
               ok = true;
               symlink_changed = false;