Re: CVS commit: src/usr.sbin/user

["Liam J. Foy" <liamfoy%sepulcrum.org@localhost>]

On Sat(30)/Jul/05 - , Jukka Salmi wrote:
> Christos Zoulas --> source-changes (2005-07-29 18:16:21 +0000):
> > 
> > Module Name:        src
> > Committed By:       christos
> > Date:               Fri Jul 29 18:16:21 UTC 2005
> > 
> > Modified Files:
> >     src/usr.sbin/user: user.c
> > 
> > Log Message:
> > Don't make it fatal if the login class doe not exist. From Jukka Salmi.
> > 
> > 
> > To generate a diff of this commit:
> > cvs rdiff -r1.83 -r1.84 src/usr.sbin/user/user.c
> 
> Hmm, this prints
> 
> useradd: Access failed for `/etc/login.conf'; will not validate class `': No 
> such file or directory
> 
> if login.conf does not exist... IMHO nothing should be printed if
> `class' is empty.
> 

I kinda disagree. At least with that error message the user knows he/she
has left the class empty. Plus, getopt should catch this out the
majority of the time.

-- 
                - Liam J. Foy
                liamfoy%sepulcrum.org@localhost