Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Date: Sat, 12 Jan 2002 04:05:06 -0800
From: Greg Stein <gstein@lyra.org>
To: Justin Erenkrantz <jerenkrantz@ebuilt.com>
Cc: dev@apr.apache.org
Subject: Re: [PATCH] Add --with-efence option.
Message-ID: <20020112040506.H15781@lyra.org>
Mail-Followup-To: Justin Erenkrantz <jerenkrantz@ebuilt.com>,
	dev@apr.apache.org
References: <20011214103336.GW1439@ebuilt.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2i
In-Reply-To: <20011214103336.GW1439@ebuilt.com>;
 from jerenkrantz@ebuilt.com on Fri, Dec 14, 2001 at 02:33:36AM -0800

On Fri, Dec 14, 2001 at 02:33:36AM -0800, Justin Erenkrantz wrote:
> I was using Electric Fence when I was debugging the pool code.
> It helped a bit, so it might be good to have it as a configure 
> option (--with-efence[=DIR]).
> 
> However, I'm wondering if we could have a GPL v. BSD disrepency
> here since efence is under GPL and we're still BSD.  So, I won't 
> commit unless someone says it is okay.

If we *required* the efence library, then we would have to release under the
GPL. But if the *user* elects to use this option, then we're okay.

This patch is totally fine to integrate, and would be a good thing for the
new pool code.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/