On 8/17/06, Garrett Rooney <rooneg@electricjellyfish.net> wrote:
> On 8/14/06, Vlad Georgescu <vgeorgescu@gmail.com> wrote:
> > I made a couple of changes to this patch to make it compile
> > successfully under win32. On Unix, we include <sasl/sasl.h>, but this
> > doesn't work on windows, we need to include <sasl.h> because we use
> > Cyrus SASL's include directory directly. We also need to define
> > STRUCT_IOVEC_DEFINED, so that sasl.h doesn't try to redefine iovec
> > (this was initially in D.J. Heap's patch, I moved it here to keep the
> > solution close to the problem). See also:
> > http://svn.haxx.se/dev/archive-2006-08/0411.shtml
>
> It looks like the only thing left to resolve is Philip's question
> about use of atexit versus pool cleanups for the sasl_done stuff.
>
> I don't personally have a strong opinion on it, although I lean
> towards agreeing with Philip since he's usually right.
>
I changed the patch to use the pool cleanup in all cases. I'm still
not 100% convinced that this is the right thing to do, but it's only a
minor issue, there's no point in debating it any further.
--
Vlad
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 17 21:34:31 2006