Re: [PATCH] add --xml to proplist command

señior ¿tyrtle? wrote:
> On 7/23/05 1:57 AM, "Julian Foad" <julianfoad@btopenworld.com> sneezed:
>
>>It is easier for me to look at your patches when you attach them with a MIME
>>type like text/plain, like you did before, rather than application.octet-stream
>
> I really have no idea what Entourage is doing, so I hope this is better (not
> that I'm doing anything different from the last two times).

Yes, this is better. Thanks.

>>I'm not sure that the ability to choosing absolute or relative paths is a
>>good idea. Just do the same as we do in the other XML outputs.
>
> Relative?

You snipped the context of my comment; the context was my interpretation of the
following line of your log message: "-verbose switches between absolute and
relative paths for xml." It made me wonder about this "verbose" option for
choosing one or the other.

I'll come back to this point when I review your latest patch and DTD.

> In this patch I'm actually parsing the data in the property to infer if its
> of seven types rather then using strictly string or data. Its important to
> do, otherwise if anyone else starts using this functionality, there's going
> to be a bunch of xsl written to accomplish the same thing.

Hmm. I'll take a look at that.

> Being said, parsing arbitray values for meaning they might not have is
> perhaps weird enough that this patch is rejected. That¹s a question.

Hmm. I'll not answer yet.

>>Thanks for taking it this far; I hope you still have time to go through
>>another round of patch and review.
>
> Theres more!?

Of course! You didn't think it would be done and finished that easily, did
you? :-)

I see you've already posted another version, with a DTD (thanks), so I'll go
and look at that.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Jul 24 02:29:33 2005