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

Re: [PATCH] Issue #1296: --force option for svn export

From: plasma <plasmaball_at_pchome.com.tw>
Date: 2003-05-23 03:53:20 CEST

On Thu, May 22, 2003 at 01:38:34PM +0100, Julian Foad wrote:
> plasma wrote:
> >
> >Sounds good. Below is another try, and it is aginst 0.23. In
> >addition, I throw SVN_ERR_WC_OBSTRUCTED_UPDATE if target is a file
> >instead of a directory in open_root() and add_directory().
>
> Please could you give a short description (in the log message and/or in the
> "--help" text) of what the "--force" option means or does. I am asking
> because issue 1296 says "nice to have a --overwrite or --force option to
> tell 'svn export' to just overwrite any files that already exist", but I
> can't see anything in this patch that overwrites files.

'svn export' barks when a directory it doesn't expect to exist exists.
For 'svn export', the first it has to do is create a directory for
destination, and it will fail if the destination directory exists in
the first place. My patch changes the behavior that svn doesn't see
an existing directory as an error if --force is given.

You don't see my patch to overwrites files because svn already does so.
All I need to do is get it past existing directories.

> Also, your original message said that your problem was that "svn co
> bogus-url" created the destination directory before failing because of the
> bad URL, and the existence of that directory caused a second attempt (with
> the correct URL) to fail. That use case could be better fixed by changing
> "svn co" to not leave a dangling directory when it fails in that way.

No, I don't say that. I never metion anything about 'svn co'. You
could find my original mails below:

http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=37578
http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=37593

And the example in the second mail is bad. William Uther gave a
better example in

http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=37596

as "untarring over an already existing dir."

Regards,
plasma

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri May 23 03:55:11 2003

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.