Re: review/comments on issue #1681

On Thu, Jan 08, 2004 at 12:05:27AM -0600, kfogel@collab.net wrote:
> I was reviewing the issue #1681 patch, and found a couple of very
> minor things. Not coding errors, but just wanted to set these down
> here so we'd remember, and have a mail to refer to from the issue.
>
> After the review is a bit of discussion.
>
> > Log message:
> >
> > Make svnserve show an error message and usage if it is run without any
> > arguments. Add a new -i/--inet option to make svnserve use
> > stdin/stdout (which was the old default). Add a new -F/--foreground
> > option to prevent svnserve from daemonizing (which is useful for
> > debugging).
>
> Typo: "--inet" should be "--inetd". We can fix if/when we apply.
>
> The "-F" option appears not to exist in the code ("--foreground" does,
> however).
[snip]

I believe Greg Hudson vetoed using a single letter for "foreground".

-- 
Michael Wood <mwood@its.uct.ac.za>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org