Re: --xml on all svn commands

Gregg Tavares wrote:
> Please excuse me if I'm breaking any kind of protocol. This is my first
> attempt at contributing.
>
> currently the --xml flag seems to only work on "svn list"

and "svn blame", and "svn info", and "svn log", and "svn status".

> Are there already plans to add it to all the other commands?

Enhancements happen as and when people decide to spend time working on them.

> Or, if not, does anyone mind if I add it and send in the patches?

Contributions welcome, *but* take time to look at what people have done so
far, and propose and discuss an XML structure on this mailing list, and
write a DTD, before starting to code.

> For exmaple, I'm using "svn lock <file>". If the file is locked it
> returns
> something like
>
> 'file:folder\file.c' is already checked out by user 'gregg' in
> 'http://svn.domain.com/project/folder/file.c'
>
> Unfortunately
>
> 1) all that changes if you are not running in English Windows so you can't
> parse by word

On unix, I'd say set LC_ALL=C, but Windows setlocale() is apparently rather
braindead, and doesn't pay attention to any environment variables :-(

> 2) filenames can have single quotes in them so trying to parse by single
> quotes won't work either.

Hmm, I wonder if we should be escaping these.

...
> Anyway, it seems like a couple of hours work to add the --xml option to
> all
> the commands if there isn't already plans. Is this a contribution I can
> make?

I think you have severely underestimated the time required, but
contributions welcome.

Take a look at
http://svn.collab.net/repos/svn/trunk/subversion/clients/cmdline/dtd/
and work on just one (or perhaps two, in the case of lock/unlock) subcommand
at a time.

Max.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Sep 27 19:19:30 2005