Re: [PATCH] Update the build system on *nix to support Cyrus SASL

On 6/5/06, Julian Foad <julianfoad@btopenworld.com> wrote:
> Julian Foad wrote:
> > Why do this now, rather than when we add the code that uses it? I'm
> > probably missing something, but we don't normally link extra libraries
> > in just because we expect to use them in the future.
>
> I've now read the rest of the weekend's mail and seen that this has already
> been asked and responded to. Let it be, as a courtesy and encouragement to
> Vlad, but let's not make a habit of it.

Thanks for the courtesy. I won't make a habit of this. For now, we're
just planning on leaving this patch in trunk, but if it turns out that
SASL support doesn't show up as planned we'll yank the build system
support. See http://svn.haxx.se/dev/archive-2006-06/0165.shtml

In future it'd make more sense to wait until the functionality is
immediately useful before committing, because in that case you won't
have to worry about the possibility of yanking it.

For Serf, we said it was OK to commit incremental (and not-yet-useful)
patches to trunk, but I now understand that this is a special
exception and does not apply as a general principle.

Cheers,

David

-- 
David James -- http://www.cs.toronto.edu/~james
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org