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

On Fri, 29 Apr 2005, Alexander Thomas wrote:

> On Fri, 2005-04-29 at 12:24 +0100, Julian Foad wrote:
> > Alexander Thomas wrote:
> > I started to implement "svn blame --xml" a while back, and ran up against this
> > problem: XML output must be in UTF-8 encoding, while the file being blamed may
> > be in an incompatible encoding. Therefore it is not possible to simply insert
> > the lines of text from the file into the XML output. Either they need
> > re-coding, which is impractical, or they should be omitted and the XML just
> > contain the blame information but not the text itself. The latter solution
> > could be useful.
> why do we need to implement xml for remaining 3 informations (rev,
> author, date)?

Because that's the essential information that blame gives you. (To be
precise, the rev is essential, but you could give the commt info as well,
since it is available and would be expensive to get.

> is it worth doing ...

If the username contains spaces, it is not easy to parse the traditional
output.

If it was not for the encoding prlbem, we would include the file contents
as well, I think. An application needing this information can just do an
svn cat to get it.

That being said, I think the XML output for svn status is moe important
than blame.

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:17:55 2005