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.9090...@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
