Bug 47114

Summary: chmod(1) manpage is wrong about -vv behavior.
Product: Documentation Reporter: Gary W. Swearingen <swear>
Component: Books & ArticlesAssignee: Giorgos Keramidas <keramida>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description Gary W. Swearingen 2003-01-15 18:10:08 UTC 
The chmod(1) manpage's -v description says that the option can be used
more than once for some special behavior, while the program doesn't care
how many times you use it.  (Confirmed by looking at
"/usr/src/bin/chmod/chmod.c".)

The -v description says the verbosity occurs "as the mode is modified",
which is easily mis-interpreted to mean "as the mode is set" or "as the
command processes the file".

================

Fix: 1) Remove the offending sentence.
2) Make the verbosity switch description more verbose.

Patch this:
.\" $FreeBSD: src/bin/chmod/chmod.1,v 1.17.2.12 2002/09/18 23:40:35 trhodes Exp $
with this:
How-To-Repeat: n/a
================
Comment 1 Giorgos Keramidas freebsd_committer freebsd_triage 2003-01-16 01:31:53 UTC 
On 2003-01-15 10:06, swear@attbi.com (Gary W. Swearingen) wrote:
> The chmod(1) manpage's -v description says that the option can be used
> more than once for some special behavior, while the program doesn't
> care how many times you use it.  (Confirmed by looking at
> "/usr/src/bin/chmod/chmod.c".)

Good catch!  But you are obviously talking about chmod.c in the
4-STABLE branch.  If you look at the source code of chmod.c in
5.0-CURRENT you will see the special behavior is there.

> Patch this:
> .\" $FreeBSD: src/bin/chmod/chmod.1,v 1.17.2.12 2002/09/18 23:40:35 trhodes Exp $

I think that the manpage is a result of an MFC (version 1.17.2.10 was
the one that introduced the changes to chmod.1) which failed to update
the source code too.  IMHO, the best thing to do would be to keep the
manpage as it is now and merge the chmod.c stuff too.

What do you think?
Comment 2 Giorgos Keramidas freebsd_committer freebsd_triage 2003-01-24 02:56:34 UTC 
State Changed
From-To: open->closed

Change 1.17.2.14, that I committed a short while ago, fixed this 
for RELENG_4.  Thanks for reporting this and helping me find out 
why the manpage was wrong.
Comment 3 Giorgos Keramidas freebsd_committer freebsd_triage 2003-01-24 02:56:34 UTC 
Responsible Changed
From-To: freebsd-doc->keramida