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

[Madan U S]

On Thursday 06 Apr 2006 3:23 am, Giovanni Bajo wrote:
> Madan U S <madan at 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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: /pipermail/svnmerge/attachments/20060406/5c845299/attachment.htm