Re: svn log improvement proposals

"Eirik Bjørsnøs" <eirbjo_at_gmail.com> writes:
> I'm attaching a patch (against trunk) that adds "<no-revprops/>" to
> mod_dav_svn, ra_neon and ra_serf.
>
> Additionally, the patch adds a --with-no-revprops to the svn log CLI.
>
> I've tested it with both neon, serf and my own Java client library and
> it seems to work fine.
>
> I must admit I haven't been coding much C since university so you
> might want to review this patch with that in mind. That being said,
> this was a pretty easy to implement given that we already have
> "all-revprops", so it shouldn't be very risky patch :-)
>
> Let me know what you think,

Very clean patch, thanks. Note that a log message always helps, see

   http://subversion.tigris.org/hacking.html#patches

which links to

   http://subversion.tigris.org/hacking.html#log-messages

In this case, we could probably write the log message ourselves if you
don't supply one.

I wasn't following the thread, so haven't applied the patch yet -- I'd
like to know that others (Eric Gillespie?) think it's a good idea first.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-07 17:58:18 CEST