Re: [PATCH] make svnmucc consistently overwrite copy targets

Philip Martin <philip_at_codematters.co.uk> wrote on 11/06/2008 03:56:02 PM:
> kmradke_at_rockwellcollins.com writes:
>
> > svnmucc doesn't really document what should happen when the copy
> > target already exists. I agree it would be better to be more
> > consistent with the svn command line.
>
> I disagree: the "svn cp URL1 URL2" command is horrible; it's often
> impossible to predict the effect unless one has exclusive access to
> the repository.

I can't commit to making it less horrible... :) I'd like
to not encourage horribleness if possible though.

> > Options I see:
> > 1) Add a --force (or maybe --overwrite) option
> > 2) Add a new action such as CPO, or CPF, OV, OVER
> >
> > Would patches for either of these be better received?
> > And if so, is #1 or #2 preferred?
>
> I don't really like either of those, although 2 is better than 1.
> svnmucc is designed to combine simple commands together, what's wrong
> with "rm URL1 cp URL2 URL1"?

This fails if URL1 didn't exist, so I need to write a wrapper script
to check to see if it exists first and then do "rm x cp x y" or
else do "cp x y"... That makes it painful for me.

Maybe a third option is to allow rm to fail silently if it doesn't
exist?

I'm flexible, I just want to implement something acceptable...

Kevin R.
Received on 2008-11-06 23:05:21 CET