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

Re: [PATCH] 'svn blame --xml' and fix for mutliple targets for svn blame

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2005-04-29 15:59:15 CEST

On Fri, 29 Apr 2005, Alexander Thomas wrote:

> On Fri, 2005-04-29 at 15:17 +0200, Peter N. Lundblad wrote:
>
> > > @@ -70,18 +131,57 @@
>
> > I think making the element optional is better than having an empty
> > string (again, see below).
> >
> > > + author ? author : " -",
> > > + time_stdout,
> >
> > This strings with padding spaces makes sense in the traditional blame
> > output, but not in XML, which is a structured format.
> >
> I agree, why not author and date to be made optional?
>
I think author is optional in list.dtd. Maybe date should be as well?
Maybe we have a bug in the list command here.

> > I think the XML output should output all information available and
> > ignore the -v flag. That's what the list command does. The output will
> > be parsed by a machine anyway.
> I don't vote for this, my suggestion is to keep both the options (-v
> and without -v) intact.
>
Why? svn list does it the way I suggest. The reason for -v is often to not
clutter the default output.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Apr 29 15:52:53 2005

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.