1.5 svn:externals and command line parsing

The new svn:externals allows externals like this, with a space between -r and
the revision:

-r 23000 http://svn.collab.net/repos/svn/trunk trunk-23000

or without the space

-r24000 http://svn.collab.net/repos/svn/trunk trunk-24000

This matches the functionality of the old externals parsing code where the space
is optional.

When trying to set a property using this on the command line, you get an error:

$ svn ps svn:externals '-r24000 http://svn.collab.net/repos/svn/trunk trunk' .
subversion/svn/main.c:1147: (apr_err=205000)
svn: Syntax error in revision argument '24000
http://svn.collab.net/repos/svn/trunk svn-trunk'

While most of the time I use svn pe to set these, having a format for an
external you cannot use via svn ps seems to be broken.

While we're on this subject, the change to support both:

URL DIR (svn <= 1.4)

and

DIR URL (svn >= 1.5 with peg revisions)

never really appealed to me. It seems needlessly complex to support these
different formats and it's one more thing to people to remember that you can
have these different formats.

I suggest just going back to

DIR URL
DIR -rN URL
DIR -r N URL

and extend URLs to have peg and relative support. The code would be simpler and
less complex UI for svn users. It would also allow 'svn ps'.

To Mike's concern about the inconsistency between the externals format and
export and checkout, that never bugged me. I thought of the externals as a
dictionary with the key being on the left that creates the directory with the
dictionary value being where to pull the directory from, so thinking as the
directory as more important than the URL.

Regards
Blair

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Aug 22 20:22:26 2007