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

Re: [PATCH] Client-side Cyrus SASL support

From: Vlad Georgescu <vgeorgescu_at_gmail.com>
Date: 2006-08-17 21:32:52 CEST

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

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.