Subject: Re: CVS commit: src/usr.sbin/user
To: source-changes@NetBSD.org, Christos Zoulas <christos@netbsd.org>
From: Liam J. Foy <liamfoy@sepulcrum.org>
List: source-changes
Date: 07/30/2005 13:27:31
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