Bug 62830

Summary: Login class isn't unused
Product: Documentation Reporter: fullermd
Component: Books & ArticlesAssignee: dannyboy <dannyboy>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description fullermd 2004-02-14 11:20:08 UTC
    Despite what passwd(5) suggests, login classes ARE actually used.
    The manpage even mentions it a few lines later  :)

How-To-Repeat:     man 5 passwd
Comment 1 Marc Silver 2004-02-16 09:07:43 UTC
Hi there,

It may be wise to mention in your patch that the login classes are not
used *by default*.  I would also add a reference to cap_mkdb(1) in the
'SEE ALSO' section, as this is required to build the login class
information.

Cheers,
Marc

-- 
Success is never final. Failure is never fatal. It is courage that
counts.  
  -- Winston Churchill
Comment 2 fullermd 2004-02-16 10:08:21 UTC
On Mon, Feb 16, 2004 at 09:07:43AM +0000 I heard the voice of
Marc Silver, and lo! it spake thus:
> 
> It may be wise to mention in your patch that the login classes are not
> used *by default*.  I would also add a reference to cap_mkdb(1) in the
> 'SEE ALSO' section, as this is required to build the login class
> information.

passwd(5) already mentions login.conf a few lines later when it's
describing the class:

     The class field is a key for a user's login class. Login classes
     are defined in login.conf(5), which is a termcap(5) style
     database of user attributes, accounting, resource, and
     environment settings.

But perhaps a cross-reference to login.conf(5) might be in order;
xref'ing cap_mkdb doesn't really make sense, since it's used to any
number of such databases.  So, this might be better:

Index: passwd.5
===================================================================
RCS file: /usr/cvs/src/share/man/man5/passwd.5,v
retrieving revision 1.40
diff -u -r1.40 passwd.5
--- passwd.5	15 Jan 2003 08:24:45 -0000	1.40
+++ passwd.5	16 Feb 2004 10:06:05 -0000
@@ -79,7 +79,7 @@
 .It gid
 User's login group id.
 .It class
-User's general classification (unused).
+User's login class.
 .It change
 Password change time.
 .It expire
@@ -311,6 +311,7 @@
 .Xr login 1 ,
 .Xr passwd 1 ,
 .Xr getpwent 3 ,
+.Xr login.conf 5 ,
 .Xr netgroup 5 ,
 .Xr adduser 8 ,
 .Xr pwd_mkdb 8 ,



-- 
Matthew Fuller     (MF4839)   |  fullermd@over-yonder.net
Systems/Network Administrator |  http://www.over-yonder.net/~fullermd/

"The only reason I'm burning my candle at both ends, is because I
      haven't figured out how to light the middle yet"
Comment 3 dannyboy freebsd_committer freebsd_triage 2004-03-21 19:37:02 UTC
State Changed
From-To: open->patched

Fixed in HEAD. 


Comment 4 dannyboy freebsd_committer freebsd_triage 2004-03-21 19:37:02 UTC
Responsible Changed
From-To: freebsd-doc->dannyboy

Fixed in HEAD.
Comment 5 dannyboy freebsd_committer freebsd_triage 2004-03-22 00:46:40 UTC
State Changed
From-To: patched->closed

These changes are already in RELENG_4.