Bug 122476

Summary: [handbook] [patch] Misleading doc for adding new users to a group
Product: Documentation Reporter: Diane Bruce <db>
Component: Books & ArticlesAssignee: freebsd-doc (Nobody) <doc>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   

Description Diane Bruce 2008-04-05 23:10:00 UTC
	3.8 is mis-leading and has resulted in a new user accidentally
	deleting himself from groups, both -M and -m should be shown
	as examples clarifying the difference.

Fix: See sgml diff



<example>
-      <title>Adding Somebody to a Group Using &man.pw.8;</title>
+      <title>Creating the list of members of a Group Using &man.pw.8;</title>

       <screen>&prompt.root; <userinput>pw groupmod teamtwo -M jru</userinput>
 &prompt.root; <userinput>pw groupshow teamtwo</userinput>
@@ -990,7 +990,7 @@
     </example>

     <para>The argument to the <option>-M</option> option is a
-      comma-delimited list of users who are members of the group.  From the
+      comma-delimited list of users who are to be in the group.  From the
       preceding sections, we know that the password file also contains a
       group for each user.  The latter (the user) is automatically added to
       the group list by the system; the user will not show up as a member
@@ -1000,6 +1000,18 @@
       <filename>/etc/group</filename> file; it will never attempt to read
       additionally data from <filename>/etc/passwd</filename>.</para>

+      <example>
+      <title>Adding a new member to a Group Using &man.pw.8;</title>
+
+      <screen>&prompt.root; <userinput>pw groupmod teamtwo -m db</userinput>
+&prompt.root; <userinput>pw groupshow teamtwo</userinput>
+teamtwo:*:1100:jru,db</screen>
+    </example>
+    <para>The argument to the <option>-m</option> option is a
+      comma-delimited list of users who are to be added to the group. Unlike
+      the previous example, these users are added to the group and do not
+      replace the list of users in the group.
+    </para>
     <example>
       <title>Using &man.id.1; to Determine Group Membership</title>--epJHGlaAdmtlpwvDKNlhrsqK5ttUMatlTycttCpIzNi3Efoc
Content-Type: text/plain; name="file.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="file.diff"

--- en_US.ISO8859-1/books/handbook/users/chapter.sgml.orig	2008-04-04 18:24:51.000000000 -0400
+++ en_US.ISO8859-1/books/handbook/users/chapter.sgml	2008-04-05 16:01:40.000000000 -0400
@@ -982,7 +982,7 @@
       the <groupname>teamtwo</groupname> group.</para>
Comment 1 dfilter service freebsd_committer freebsd_triage 2008-04-24 17:17:55 UTC
roam        2008-04-24 16:17:44 UTC

  FreeBSD doc repository

  Modified files:
    en_US.ISO8859-1/books/handbook/users chapter.sgml 
  Log:
  Clarify the use of the "-M" and "-m" options to "pw groupmod" in
  the "Groups" section.  Specifically, explain that "-M" sets the group
  memberlist to the exact userlist specified, while "-m" adds to
  the current list.
  
  PR:             122476
  Submitted by:   db
  
  Revision  Changes    Path
  1.59      +15 -2     doc/en_US.ISO8859-1/books/handbook/users/chapter.sgml
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 2 Peter Pentchev freebsd_committer freebsd_triage 2008-04-24 17:24:29 UTC
State Changed
From-To: open->closed

I've just committed a slightly modified version of your patch. 
Thanks!