[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn log improvement proposals

From: Eirik Bjørsnøs <eirbjo_at_gmail.com>
Date: Sun, 14 Sep 2008 19:32:00 +0200

> +1: This and your original patch look good, but I have only read
> them. Unfortunately, my participation is email-only these days.
>
> If no one commits it within a few days, please put it in the
> issue tracker, and reference this thread.

Eric,

I've added an issue for this as requested:

http://subversion.tigris.org/issues/show_bug.cgi?id=3286

I've attached the combined patch (the code plus the one for
notes/webdav-protocol) and added the log message to that issue.

I'm including a copy of the log message here, hoping that might entice
someone into committing it :-)

[[[
Add new command line switch "--with-no-revprops" to the svn log CLI.
Extend the DAV protocol with a "no-revprops" flag and update ra_serf
and ra_neon to send this flag if the revprops array is empty. This
brings the DAV protocol on par with the SVN protocol and exposes this
functionality to command line users.

* subversion/mod_dav_svn/reports/log.c
 (dav_svn__log_report): Read "no-revprops" and leave an empty array
of rev-props to fetch.

* subversion/libsvn_ra_serf/log.c
 (svn_ra_serf__get_log): Add "<no-revprops/>" to log report request
if revprops array is empty.

* subversion/libsvn_ra_neon/log.c
 (svn_ra_neon__get_log): Add "<no-revprops/>" to log report request
if revprops array is empty.

* subversion/svn/cl.h
 (svn_cl__opt_state_t): Add no_revprops

* subversion/svn/log-cmd.c
 (svn_cl__log): Add check for --with-no-revprops only being used in XML mode
 (svn_cl__log): Create an empty revprops array if no_revprops is set.

* subversion/svn/main.c
 (svn_cl__longopt_t) Add opt_with_no_revprops
 (svn_cl__options) Add "with-no-revprops" option
 (main) Check for opt_with_no_revprops being set

* notes/webdav-protocol
 (log-report) Add "no-revprops" option

]]]

Thanks,
Eirik.

---------------------------------------------------------------------
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-14 19:32:33 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.