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

Re: svn commit: r1087321 - /subversion/trunk/subversion/svn/switch-cmd.c

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Thu, 31 Mar 2011 10:53:56 -0400

On 03/31/2011 10:46 AM, stsp_at_apache.org wrote:
> Author: stsp
> Date: Thu Mar 31 14:46:16 2011
> New Revision: 1087321
>
> URL: http://svn.apache.org/viewvc?rev=1087321&view=rev
> Log:
> * subversion/svn/switch-cmd.c
> (svn_cl__switch): Convert path to local style in error message about
> non-related switch source and target. Tweak wording explaining the
> --ignore-ancestry override.

Thanks for the local-style change.

I'm not sure I like the new wording, though. (In fact, I specifically avoid
it myself.) There are other things which could prevent the operation from
running, so I don't like the implication that this option is a magic potion
which overcomes all such barriers. I'd rather be specific -- if you pass
--ignore-ancestry, the ancestry won't be considered when deciding if this is
a valid request.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2011-03-31 16:54:31 CEST

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