Return-Path: Delivered-To: apmail-httpd-test-dev-archive@httpd.apache.org Received: (qmail 54398 invoked by uid 500); 27 Mar 2002 00:32:42 -0000 Mailing-List: contact test-dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: test-dev@httpd.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list test-dev@httpd.apache.org Received: (qmail 54380 invoked from network); 27 Mar 2002 00:32:42 -0000 X-Authentication-Warning: cancer.clove.org: jerenk set sender to jerenkrantz@apache.org using -f Date: Tue, 26 Mar 2002 16:32:49 -0800 From: Justin Erenkrantz To: test-dev@httpd.apache.org Subject: Re: cvs commit: httpd-test/flood/build apr_common.m4 Message-ID: <20020326163249.A14500@apache.org> Mail-Followup-To: Justin Erenkrantz , test-dev@httpd.apache.org References: <20020325225820.40403.qmail@icarus.apache.org> <20020325152930.K1154@clove.org> <20020325153427.D1000@apache.org> <20020325183741.N1154@clove.org> <20020326091951.B5820@apache.org> <20020326161356.T1154@clove.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <20020326161356.T1154@clove.org>; from aaron@clove.org on Tue, Mar 26, 2002 at 04:13:56PM -0800 X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N On Tue, Mar 26, 2002 at 04:13:56PM -0800, Aaron Bannert wrote: > On Tue, Mar 26, 2002 at 09:19:51AM -0800, Justin Erenkrantz wrote: > > > If it's not installed as part of APR it should be. We shouldn't have it in > > > our repository merely because we don't want to have to keep them in sync. > > > > It wouldn't make sense as part of the install as it is only needed > > to build configure. > > I disagree. You can't have it both ways. If flood, an application that > depends on an already-installed version of APR, requires some autoconf > macros provided by APR then those macros must be installed by APR. Uh, how can an installation of APR provide the macros? The paths are hardcoded in flood's configure.in. To be clear, we're talking about this line in configure.in: dnl m4 Macros from APR sinclude(build/apr_common.m4) We have to have this file when we *generate* configure or autoconf will die. We can't rely on installed versions of APR since we don't know where these are and we have no way of telling autoconf that. We also shouldn't have the user change that path to point at their installed location of APR manually. Therefore, this file needs to exist somewhere in *our* tree. I think you're not understanding the problem here. httpd will have this problem too. And, the only way I can see this happening is to have its own copies of the macros. httpd-2.0 cheats by enforcing that apr must be in the source tree. -- justin