cvs commit: src/contrib/cvs/src server.c

[Xin LI]

> -----Original Message-----
> From: owner-cvs-all at freebsd.org 
> [mailto:owner-cvs-all at freebsd.org] On Behalf Of Scott Long
> Sent: Monday, January 26, 2004 12:21 PM
> To: Alfred Perlstein
> Cc: Jacques A. Vidrine; cvs-src at freebsd.org; 
> src-committers at freebsd.org; re at freebsd.org; cvs-all at freebsd.org
> Subject: Re: cvs commit: src/contrib/cvs/src server.c
> 
> Alfred Perlstein wrote:
> > * Jacques A. Vidrine <nectar at FreeBSD.org> [040125 16:09] wrote:
[commit log omitted]
> > 
> > Nice of us to break something that used to work without a 
> > deprication/warning period. :(
> > 
> > Note: this is what my earlier mail was about.
> > 
> 
> We've had quite a few people request that be put into 5.2.1.  
> I didn't see any dissenting email from you, can you elaborate?
I think he may mean the configuration in /etc/inetd.conf, circa line 63,
where the example shows how to run cvs pserver as root.

Personally I think it's a bad idea to run cvs pserver as root, as pserver
does not special privileges that only root have

Maybe we should change it to 'nobody'?

Well, whatever, I'd admit Alfred's statements is somewhat correct, but I'd
support Jacques' change, and it might be better to see a HEADSUP in
current@?

Cheers,
Xin LI