2008/7/4 Julian Foad <julianfoad_at_btopenworld.com>:
> On Wed, 2008-07-02 at 18:06 -0400, Karl Fogel wrote:
>> Julian Foad <julianfoad_at_btopenworld.com> writes:
>> > It's not a library function, it's local to the "svn" client. I didn't
>> > think it was quite worth moving it into a library for this amount of
>> > duplication. Want me to? (Uh, libsvn_client/cmdline.c ?)
>>
>> Oh, my bad, I got confused between 'svn' and 'svnlook'.
>>
>> But yeah, I think it's probably best to share it, via cmdline.c.
>
> OK, here's a go at factoring out this print-a-property-as-XML function
> "svn_cl__print_xml_prop()" function so that it can be shared with
> "svnlook", by moving it into libsvn_client and calling it
> "svn_client__print_xml_prop()".
>
> I didn't put it in "cmdline.c" because it's not actually doing
> command-line output, just forming XML in a string buffer, and generating
> XML-formatting output is not specific to command-line clients.
>
> So it's in a new file "subversion/libsvn_client/xml.c".
>
> Making "svnlook" use this adds a dependency of svnlook on libsvn_client,
> which I don't see as a problem, but I'd better mention it.
Ugh. Why not in libsvn_subr/cmdline.c or libsvn_subr/xml.c? Making a
server-side program depend on client libraries (including the RA libs,
which by default starts to drag in Neon etc) isn't so great.
--dave
>
> A review, anyone, please?
>
> - Julian
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
> For additional commands, e-mail: dev-help_at_subversion.tigris.org
>
--
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-07-08 05:42:08 CEST