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

RE: [Svnmerge] [PATCH] svnmerge.py: Add check for already init-ed head

From: Madan U S <madan_at_collab.net>
Date: 2006-04-06 08:24:44 CEST

On Thursday 06 Apr 2006 3:23 am, Giovanni Bajo wrote:
> Madan U S <madan@collab.net> wrote:
> > [[[
> > Add check for already initialized head.
> >
> > * contrib/client-side/svnmerge.py
> > (action_init): Modified to print error message and exit,
> > when the branch provided has already been initialized for
> > the given head.
> > ]]]
>
> This is commonly used to "reset" the merge status after some confusing
> commands.

oh, no. If it is meant to be used like that, it should not be called 'init'.
Maybe 'sync' or 'integrate'?

> At the very least, you should make --force override this check.

agree.

> I'd be more satisfied if you just printed a warning and let the operation
> continue: the user can easily revert the command after the fact if he wants
> to.

Revert may not be possible in some cases, given the other user changes.
I'd rather like the command failing with a message saying 'Use --force to
force this operation'.

Regards,
Madan.
Received on Thu Apr 6 08:25:43 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.