You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by "William A. Rowe, Jr." <wr...@rowe-clan.net> on 2006/03/31 23:02:13 UTC

When did this break?

./configure --with-apr=srclib/apr

configure:5784: checking for APR version 1.2.0 or later
conftest.c:20:25: apr_version.h: No such file or directory
configure:5812: result: no
configure:5818: error: APR version 1.2.0 or later is required

This test is bogus because apr_version includes apr.h and that doesn't
exist in a source-build tree.  Why not test the file with awk?

Bill

Re: When did this break?

Posted by Garrett Rooney <ro...@electricjellyfish.net>.
On 3/31/06, William A. Rowe, Jr. <wr...@rowe-clan.net> wrote:
> It seems this can be trivially solved by ensuring we defer this test until
> after we invoke the sub-configure of apr and apr-util.
>
> Comments?

Makes sense to me.

-garrett

Re: When did this break?

Posted by Joe Orton <jo...@redhat.com>.
On Mon, Apr 03, 2006 at 08:53:24AM +0100, Joe Orton wrote:
> What's really needed here is a "--with-included-apr"-type flag which 
> forces use of the bundled sources over installed versions, rather than 
> trying to overload --with-apr to do that.

e.g. this, slightly hacky but does it solve the problem you were having?

Index: configure.in
===================================================================
--- configure.in	(revision 388210)
+++ configure.in	(working copy)
@@ -61,8 +61,16 @@
 
 echo $ac_n "${nl}Configuring Apache Portable Runtime library ...${nl}"
 
-APR_FIND_APR("$srcdir/srclib/apr", "./srclib/apr", 1, 1)
+AC_ARG_WITH(included-apr,
+APACHE_HELP_STRING(--with-included-apr,Use bundled copies of APR/APR-Util))
 
+if test "x$with_included_apr" = "xyes"; then
+   apr_found=reconfig
+   apr_config="$srcdir/srclib/apr/apr-1-config"
+else 
+   APR_FIND_APR("$srcdir/srclib/apr", "./srclib/apr", 1, 1)
+fi
+
 if test "$apr_found" = "no"; then
   AC_MSG_ERROR([APR not found.  Please read the documentation.])
 fi
@@ -89,7 +97,12 @@
 
 echo $ac_n "${nl}Configuring Apache Portable Runtime Utility library...${nl}"
 
-APR_FIND_APU("$srcdir/srclib/apr-util", "./srclib/apr-util", 1, 1)
+if test "x$with_included_apr" = "xyes"; then
+   apu_found=reconfig
+   apu_config="${srcdir}/srclib/apr-util/apu-1-config"
+else 
+   APR_FIND_APU("$srcdir/srclib/apr-util", "./srclib/apr-util", 1, 1)
+fi
 
 if test "$apu_found" = "no"; then
   AC_MSG_ERROR([APR-util not found.  Please read the documentation.])




Re: When did this break?

Posted by Joe Orton <jo...@redhat.com>.
On Fri, Mar 31, 2006 at 03:23:11PM -0600, William Rowe wrote:
> It seems this can be trivially solved by ensuring we defer this test until
> after we invoke the sub-configure of apr and apr-util.

The "sufficient version" test is currently only used if the 
sub-configure of ap[ru] is *not* invoked - when $apr_found is "yes", not 
"reconfig".

"--with-apr=srclib/apr" actually treats the bundled sources like any 
"external" copy of APR, so must be already-configured at the time the 
httpd configure is run, AFAIK.  If that's not happened, the apX-N-config 
file would not be found, so configure should fail earlier.

What's really needed here is a "--with-included-apr"-type flag which 
forces use of the bundled sources over installed versions, rather than 
trying to overload --with-apr to do that.

joe

> William A. Rowe, Jr. wrote:
> >./configure --with-apr=srclib/apr
> >
> >configure:5784: checking for APR version 1.2.0 or later
> >conftest.c:20:25: apr_version.h: No such file or directory
> >configure:5812: result: no
> >configure:5818: error: APR version 1.2.0 or later is required
> >
> >This test is bogus because apr_version includes apr.h and that doesn't
> >exist in a source-build tree.  Why not test the file with awk?
> >
> >Bill
> >
> >.
> >

joe

Re: When did this break?

Posted by "William A. Rowe, Jr." <wr...@rowe-clan.net>.
It seems this can be trivially solved by ensuring we defer this test until
after we invoke the sub-configure of apr and apr-util.

Comments?

Bill

William A. Rowe, Jr. wrote:
> ./configure --with-apr=srclib/apr
> 
> configure:5784: checking for APR version 1.2.0 or later
> conftest.c:20:25: apr_version.h: No such file or directory
> configure:5812: result: no
> configure:5818: error: APR version 1.2.0 or later is required
> 
> This test is bogus because apr_version includes apr.h and that doesn't
> exist in a source-build tree.  Why not test the file with awk?
> 
> Bill
> 
> .
>