Re: svn commit: r1659013 - /subversion/trunk/subversion/libsvn_subr/dso.c

Ivan Zhakov <ivan_at_visualsvn.com> writes:

> Please understand my correctly: I'll be glad see Subversion doesn't
> require single-thread initialize functions, but I don't see how is it
> possible in reliable way. Currently the only reliable solution is to
> call svn_dso_initialize2() as first call after apr_initialize().

That may not be reliable if an unmanged pool is passed to the FS layer.

In the end there are rules for using pools with Subversion. One rule is
that pools passed to the RA/FS layers need to be cleared before the DSO
is unloaded. Calling svn_dso_initialize2 early is a way to do that in
lots of cases. Explicitly clearing all pools passed to RA/FS also
works.

To claim that calling svn_dso_initialize2 as the first call after
apr_initialize is the only solution is impractical: mod_dav_svn is never
going to be able to do that.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*