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

Re: [PATCH]operational logging for checkout and update should log revnum too.

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: 2007-01-04 19:57:12 CET

Kamesh Jayachandran wrote:
> Hi All,
> Find the attached patch and log.
>
> With regards
> Kamesh Jayachandran

Ping...

Anybody had a chance to look at this yet? I'll file an issue if nothing
happens in the next few days.

-Hyrum

> ------------------------------------------------------------------------
>
> [[[
>
> Checkout/export and Update should should log the revnum too.
>
> * subversion/mod_dav_svn/reports/update.c
> (dav_svn__update_report):
> Log revnum for checkout/export and update.
>
> Patch by: kameshj
> Suggested by: Honey George <george@collab.net>
>
> ]]]
>
>
> ------------------------------------------------------------------------
>
> Index: subversion/mod_dav_svn/reports/update.c
> ===================================================================
> --- subversion/mod_dav_svn/reports/update.c (revision 22777)
> +++ subversion/mod_dav_svn/reports/update.c (working copy)
> @@ -1280,15 +1280,16 @@
> reports it (and it alone) to the server as being empty. */
> if (entry_counter == 1 && entry_is_empty)
> action = apr_psprintf(resource->pool,
> - "checkout-or-export '%s'",
> - svn_path_uri_encode(spath, resource->pool));
> + "checkout-or-export '%s' r%" SVN_REVNUM_T_FMT,
> + svn_path_uri_encode(spath, resource->pool),
> + revnum);
> else
> {
> if (text_deltas)
> action = apr_psprintf(resource->pool,
> - "update '%s'",
> - svn_path_uri_encode(spath,
> - resource->pool));
> + "update '%s' r%" SVN_REVNUM_T_FMT,
> + svn_path_uri_encode(spath, resource->pool),
> + revnum);
> else
> action = apr_psprintf(resource->pool,
> "remote-status '%s'",

Received on Thu Jan 4 19:57:20 2007

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