[Modcplusplus-user] Re: mod_cplusplus thanks

[John K. S. <jo...@st...>]

Hi Will -

It turns out even tthe default RedHat installation of Apache has a 
similar layout - I tend to work from CVS or downloading, so I hadn't 
caught it.  I have applied your changes to allow for --with-apr, plus 
an attempt at sniffing out the alternative.. e.g. path 
/path/include/apache2 -

I plan on trying to install a couple of distros (rh9, etc) and trying 
it out, but for the time being I've committed and will issue a point 
release.

Thanks for all the input - let me know if you have any more ideas.  
feel free to try out building from cvs.

John

On May 7, 2004, at 2:25 PM, William F. Dowling wrote:

> John K. Sterling writes:
>> Ahh... so it had a hard time finding apr... I think I've seen that on
>> other default installations before.  I could probably just make the
>> search path smarter... let me look at the build scripts.  As for the
>> test suite, it looks like the test suite is broken, not the module.
>> I'll double check and fix it tonight.  I'm due for an updated release.
>>
>> Thanks -
>>
>> sterling
>
> FWIW here is the patch I made to configure.in so that it builds
> Makefiles I can use directly.  I never used autoconf before so this
> might not be the best way to do it.  This patch just handles the
> problems of:
> - autoheader gives error message
>      autoheader2.50: error: AC_CONFIG_HEADERS not found in configure.in
> - CPPFLAGS need to have -I<path-to-apr.h>.  Now run like this:
>      ./autogen.sh --with-httpd=/usr/share/apache2 
> --with-apr_path=/usr/include/apr-0
> - better prettyprint from ./configure --help
>
> It does not handle the other issues I mentioned that I have worked
> around by adding symlinks under /usr/share/apache2 on my system.
>
> Hope this is of use.
>
> Will
>
> $ diff -u ../mod_cplusplus/configure.in configure.in
> --- ../mod_cplusplus/configure.in       2003-08-19 21:36:54.000000000 
> -0400
> +++ configure.in        2004-05-07 14:08:09.000000000 -0400
> @@ -5,6 +5,7 @@
>  AC_PROG_CXX
>  AM_PROG_LIBTOOL
>
> +AC_CONFIG_HEADERS(config.h)
>  dnl hack to get shlib extension
>  AC_MSG_CHECKING(what the shared library extension is)
>  SHLIB_EXT=`echo /usr/lib/libc.* | sed -e 's@.*\.@@'`
> @@ -12,7 +13,7 @@
>  AC_SUBST(SHLIB_EXT)
>
>  AC_MSG_CHECKING(for httpd)
> -AC_ARG_WITH(httpd,   [  --with-httpd Specify path to httpd source 
> directory ],
> +AC_ARG_WITH(httpd,   [  --with-httpd            Specify path to httpd 
> source directory ],
>  [
>         if test ! -f $withval/build/config_vars.mk; then
>                 AC_MSG_ERROR($withval not a valid source or shadow dir)
> @@ -38,6 +39,18 @@
>         AC_MSG_ERROR(--with-httpd not given)
>  ])
>
> +AC_MSG_CHECKING(for apr.h)
> +AC_ARG_WITH(apr_path,   [  --with-apr_path         Specify path to 
> apr include directory ],
> +[
> +       if test ! -f $withval/apr.h; then
> +               AC_MSG_ERROR($withval not a valid path to apr.h)
> +       else
> +               CPPFLAGS="$CPPFLAGS -I$withval"
> +               AC_MSG_RESULT(found $withval/apr.h)
> +       fi
> +],)
> +
> +
>  AC_CHECK_LIB(stdc++, __gxx_personality_v0, LIB_STDCPP="-lstdc++")
>      if test "$LIB_STDCPP" == "" ; then
>         CPPFLAGS="$CPPFLAGS -DNO_STDCPP"
>
>
> ______________________________________________________________________
> This email has been scanned by the MessageLabs Email Security System.
> For more information please visit http://www.messagelabs.com/email
> ______________________________________________________________________
>