On Thu, 01 Mar 2007, Stefan Küng wrote:
> The svn_client_cat2() function currently doesn't work correctly if the
> revision is specified as svn_opt_revision_working. In the main function,
> there's a check for BASE, COMMITTED and UNSPECIFIED revisions to do a
> local cat, but that check doesn't use WORKING.
> The function cat_local_file() however correctly uses
> svn_opt_revision_working to do the job (but of course it's never called
> that way).
> Handle svn_opt_revision_working properly in svn_client_cat2().
> * subversion/libsvn_client/cat.c : Allow svn_opt_revision_working
> Index: subversion/libsvn_client/cat.c
> --- subversion/libsvn_client/cat.c (revision 23293)
> +++ subversion/libsvn_client/cat.c (working copy)
> @@ -176,9 +176,11 @@
> if (! svn_path_is_url(path_or_url)
> && (peg_revision->kind == svn_opt_revision_base
> + || peg_revision->kind == svn_opt_revision_working
> || peg_revision->kind == svn_opt_revision_committed
> || peg_revision->kind == svn_opt_revision_unspecified)
> && (revision->kind == svn_opt_revision_base
> + || revision->kind == svn_opt_revision_working
> || revision->kind == svn_opt_revision_committed
> || revision->kind == svn_opt_revision_unspecified))
Thanks Stefan, looks good.
Running tests on it now, with plans to commit...
Received on Sat Mar 3 00:38:02 2007
- application/pgp-signature attachment: stored