Pl. find attached the patch with the --force option change.
[[[
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 check is skipped on using the --force
option.
]]]
Regards,
Madan.
PS : Apologize for top-posting.
-----Original Message-----
From: Madan U S [mailto:madan@collab.net]
Sent: Thu 4/6/2006 11:54 AM
To: Giovanni Bajo; dev@subversion.tigris.org; svnmerge@orcaware.com
Subject: RE: [Svnmerge] [PATCH] svnmerge.py: Add check for already init-ed head
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.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Apr 6 11:51:28 2006