> +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