David Kimdon writes:
> Hi,
>
> Here is a patch for the import issues indicated below. Any comments
> and/or criticism is, of course, welcome. I'm particularly interested
> in comments on the change I made to get_ra_editor(). I need something
> like 'stat()' for the repository, and I believe that this extra error
> checking in get_ra_editor() does no harm, but maybe I missed
> something. The change to subversion/libsvn_client/commit.c(import) is
> similar to that which has been reviewed by cmpilato previously.
>
> -David
>
> Fix issue #933: svn import syntax is confusing; merge 1st and 3rd arguments
> Fix issue #735: import should 'mkdir -p'
>
> Import syntax is now:
>
> svn import [PATH] URL
>
I was going to complain about the switching of the PATH and URL
arguments, then I realised that 'import' now matches 'cp', which is
good.
+1
> Parent directories in URL are created as necessary. Testsuite and
> documentation updated as well.
Hmm, what happens if I do 'svn import foo/bar file://somewhere/repo/'?
Do we end up with 'file://somewhere/repo/foo/bar'?
Robert
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jun 2 11:39:37 2003