You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Stefan Sperling <st...@elego.de> on 2008/04/28 21:47:05 UTC
Re: svn commit: r30685 -
branches/svnserve-logging/subversion/svnserve
On Mon, Apr 28, 2008 at 01:37:35PM -0700, epg@google.com wrote:
> I suppose the lack of response means this has already been
> reviewed, isn't going to be reviewed, or may get more attention
> on trunk ;->. I'll commit today or tomorrow, and then commit
> some error logging changes I have pending.
I looked over the diff very briefly, but for lack of time didn't
look through it as thoroughly as I was planning to.
I didn't want to send a mail pointing out the single thing I noticed
while skimming over it, but now that you're gonna merge anyway,
and before it gets lost:
Index: subversion/libsvn_subr/log.c
===================================================================
--- subversion/libsvn_subr/log.c (.../trunk) (revision 0)
+++ subversion/libsvn_subr/log.c (.../branches/svnserve-logging) (revision 30692)
@@ -0,0 +1,379 @@
+/*
+ * serve.c : Functions for serving the Subversion protocol
^^^^^^^
Should be log.c, right?
--
Stefan Sperling <st...@elego.de> Software Monkey
German law requires the following banner :(
elego Software Solutions GmbH HRB 77719
Gustav-Meyer-Allee 25, Gebaeude 12 Tel: +49 30 23 45 86 96
13355 Berlin Fax: +49 30 23 45 86 95
http://www.elego.de CEO: Olaf Wagner
Store password unencrypted (yes/no)? No
Re: svn commit: r30685 - branches/svnserve-logging/subversion/svnserve
Posted by Eric Gillespie <ep...@pretzelnet.org>.
Stefan Sperling <st...@elego.de> writes:
> + * serve.c : Functions for serving the Subversion protocol
> ^^^^^^^
>
> Should be log.c, right?
Fixed in r30830, thanks.
--
Eric Gillespie <*> epg@pretzelnet.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org