Re: [PATCH] Extend error message for url demanding wc rev, to all subcommands

I (Julian Foad) wrote:
> Kannan wrote:
> > At present when invalid revision keywords (BASE|PREV|COMMITTED) are used
> > with a URL, svn gracefully errors out only for four subcommands (merge,
> > diff, copy, log AFAIK) as of now. This patch extends to other
> > subcommands too, by having the error message in a common spot.
>
> Excellent. We needed this. Thank you.
>
> [...]
> > P.S: Part of the changes made in ra.c and merge.c, is also submitted in
> > [1]. This patch extends the same.
> >
> > [1]-http://mail-archives.apache.org/mod_mbox/subversion-dev/200912.mbox/<4B1678AC.9090403@collab.net>
>
> First, I'll review and commit that patch, as it is fixing bugs.

(I committed something like that earlier patch in r887187.)

Thanks for this new patch. Looks good. Committed in r887205.

- Julian
Received on 2009-12-04 15:16:45 CET