Re: [PATCH]: notification progress improvement/fixes (issue #901)

David Anderson wrote:

> Branko ÄŒibej wrote:
>
>> Uh? Where does this dependency come from? I don't see either svnserve
>> or libsvn_ra_svn using this constructor anywhere.
>
> subversion/libsvn_ra_svn/client.c includes
> subversion/libsvn_ra/wrapper_template.h, which refers to the
> constructor in compat_open().

Ah. Hm. Having svnserve delend on libsvn_ra seems ... unclean, somehow.
Oh well.

> Any further comments before I submit a (final?) followup patch for
> this fix?

I posted a few comments about the r16010 commit. Other than that, no.
Nice work, Dave an Stefan; thanks!

-- Brane

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Aug 31 04:24:19 2005