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

Re: [Patch] Add '--with-revprop' to CHANGES file

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Fri, 16 May 2008 14:22:12 -0400

Eric Gillespie wrote:
> "Bert Huijben \(@home\)" <bert_at_vmoo.com> writes:
>
>> I don't understand why this value is passed as a value in svn_client_ctx_t
>> instead of using a argument to svn_client_*() in the c api (We revved all
>> these functions in 1.5 anyway).
>>
>> It doesn't seem logical that I want to set the same extra revprops on every
>> command executed during the lifetime of my svn_client_ctx_t. (Many c-api
>> users don't recreate the client ctx for every command they use).
>
> I agree this was a mistake; I never use svn_client, so never
> noticed. You've caught it too late, I'm afraid.

Has he? Seems like a pretty straightforward change to these
not-yet-released APIs, don't you think? I'd certainly consider it minor
enough to not force a reset of the soak time.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2008-05-16 20:23:27 CEST

This is an archived mail posted to the Subversion Dev mailing list.