On 26 October 2015 at 13:00, Bert Huijben <b...@qqmail.nl> wrote: >> -----Original Message----- >> From: Ivan Zhakov [mailto:i...@visualsvn.com] >> Sent: maandag 26 oktober 2015 09:05 >> To: dev@subversion.apache.org; Bert Huijben <b...@qqmail.nl> >> Subject: Re: svn commit: r1705843 - in /subversion/trunk/subversion: >> libsvn_client/externals.c tests/cmdline/externals_tests.py > > >> ]]] >> See attached patch. >> >> Otherwise error message could be confusing, in some cases. For example >> if svn_wc__external_remove() return SVN_ERR_CANCELED error message >> will be something like: >> [[[ >> The external '%s' defined in %s at '%s' cannot be checked out because >> '%s' is already a versioned path. >> The operation was interrupted >> ]]] > > I didn't really answer your mail: > > + 0.5 on the patch. > > I think either way would work. If you like to see it changed before the > backport, feel free to add my +1 on your patch. > I've committed my patch in r1710558 and going to add to backport now.
-- Ivan Zhakov