You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Ben Laurie <be...@algroup.co.uk> on 1999/10/09 22:56:43 UTC

Re: cvs commit: apache-2.0/src Configure

manoj@hyperreal.org wrote:
> 
> manoj       99/10/09 13:28:41
> 
>   Modified:    src      Configure
>   Log:
>   Make Configure run autoheader in the APR directory.
> 
>   Revision  Changes    Path
>   1.9       +1 -1      apache-2.0/src/Configure
> 
>   Index: Configure
>   ===================================================================
>   RCS file: /home/cvs/apache-2.0/src/Configure,v
>   retrieving revision 1.8
>   retrieving revision 1.9
>   diff -u -d -u -r1.8 -r1.9
>   --- Configure 1999/10/01 16:22:48     1.8
>   +++ Configure 1999/10/09 20:28:40     1.9
>   @@ -1727,7 +1727,7 @@
>    if [ -d ./lib/apr ]; then
>        if [ ! -f ./lib/apr/configure ]; then
>            echo " + bootstrapping Apache Portable Runtime (APR)"
>   -        (cd lib/apr && autoconf)
>   +        (cd lib/apr && autoconf && autoheader)

Ah. This really ought to be triggered by the existence of autoconf, or
it doesn't actually help much (after the first time), does it?

Cheers,

Ben.

--
http://www.apache-ssl.org/ben.html

"My grandfather once told me that there are two kinds of people: those
who work and those who take the credit. He told me to try to be in the
first group; there was less competition there."
     - Indira Gandhi

Re: cvs commit: apache-2.0/src Configure

Posted by Sascha Schumann <sa...@schumann.cx>.
    Hi to the list. 

On Sun, Oct 10, 1999 at 02:13:25PM -0500, Manoj Kasichainula wrote:
> On Sat, Oct 09, 1999 at 11:50:17PM +0100, David Reid wrote:
> > Also I normally use autoheader first.  Does it make any difference?
> 
> I know next to nothing about autoconf. If the ordering should
> be switched, I sure won't complain, but it seems to work for me.

    Sure, because it doesn't make any difference. autoheader
    creates the config.h.in template, while autoconf creates
    configure. Dependencies (taken from [1]):

        $(config_h_in): configure.in acconfig.h
            autoheader

        configure: configure.in aclocal.m4
            autoconf

    [1] http://cvs.php.net/cvsweb.cgi/build.mk?rev=1.6&cvsroot=php4

-- 

          Regards,

                            Sascha Schumann
                                 Consultant

Re: cvs commit: apache-2.0/src Configure

Posted by Manoj Kasichainula <ma...@io.com>.
On Sat, Oct 09, 1999 at 11:50:17PM +0100, David Reid wrote:
> Also I normally use autoheader first.  Does it make any difference?

I know next to nothing about autoconf. If the ordering should
be switched, I sure won't complain, but it seems to work for me.

-- 
Manoj Kasichainula - manojk at io dot com - http://www.io.com/~manojk/

Re: cvs commit: apache-2.0/src Configure

Posted by David Reid <ab...@dial.pipex.com>.
Also I normally use autoheader first.  Does it make any difference?

d.
---- Original Message ----- 
From: Ben Laurie <be...@algroup.co.uk>
To: <ne...@apache.org>
Cc: <ap...@apache.org>
Sent: 09 October 1999 21:56
Subject: Re: cvs commit: apache-2.0/src Configure


> manoj@hyperreal.org wrote:
> > 
> > manoj       99/10/09 13:28:41
> > 
> >   Modified:    src      Configure
> >   Log:
> >   Make Configure run autoheader in the APR directory.
> > 
> >   Revision  Changes    Path
> >   1.9       +1 -1      apache-2.0/src/Configure
> > 
> >   Index: Configure
> >   ===================================================================
> >   RCS file: /home/cvs/apache-2.0/src/Configure,v
> >   retrieving revision 1.8
> >   retrieving revision 1.9
> >   diff -u -d -u -r1.8 -r1.9
> >   --- Configure 1999/10/01 16:22:48     1.8
> >   +++ Configure 1999/10/09 20:28:40     1.9
> >   @@ -1727,7 +1727,7 @@
> >    if [ -d ./lib/apr ]; then
> >        if [ ! -f ./lib/apr/configure ]; then
> >            echo " + bootstrapping Apache Portable Runtime (APR)"
> >   -        (cd lib/apr && autoconf)
> >   +        (cd lib/apr && autoconf && autoheader)
> 
> Ah. This really ought to be triggered by the existence of autoconf, or
> it doesn't actually help much (after the first time), does it?
> 
> Cheers,
> 
> Ben.
> 
> --
> http://www.apache-ssl.org/ben.html
> 
> "My grandfather once told me that there are two kinds of people: those
> who work and those who take the credit. He told me to try to be in the
> first group; there was less competition there."
>      - Indira Gandhi


Re: cvs commit: apache-2.0/src Configure

Posted by Ben Laurie <be...@algroup.co.uk>.
Manoj Kasichainula wrote:
> 
> On Sat, Oct 09, 1999 at 09:56:43PM +0100, Ben Laurie wrote:
> > manoj@hyperreal.org wrote:
> > >   --- Configure 1999/10/01 16:22:48     1.8
> > >   +++ Configure 1999/10/09 20:28:40     1.9
> > >   @@ -1727,7 +1727,7 @@
> > >    if [ -d ./lib/apr ]; then
> > >        if [ ! -f ./lib/apr/configure ]; then
> > >            echo " + bootstrapping Apache Portable Runtime (APR)"
> > >   -        (cd lib/apr && autoconf)
> > >   +        (cd lib/apr && autoconf && autoheader)
> >
> > Ah. This really ought to be triggered by the existence of autoconf, or
> > it doesn't actually help much (after the first time), does it?
> 
> In my sleepy haze (I work on Hawaiian time on weekends), I can't quite
> parse this. Are you saying that we should be checking for the
> existence of autoconf first so that error messages are sane?
> 
> I agree. I'm not sure whether it's worth the effort to fix up
> Configure too much if we're placing the whole ball with autoconf. I'm
> probably going to make autoconf my next project, so hopefully this
> isn't too far away.

No, what I mean is that if you have autoconf, you should _always_ run
it, not just when there's no configure. I've fixed it anyway.

Cheers,

Ben.

--
http://www.apache-ssl.org/ben.html

"My grandfather once told me that there are two kinds of people: those
who work and those who take the credit. He told me to try to be in the
first group; there was less competition there."
     - Indira Gandhi

Re: cvs commit: apache-2.0/src Configure

Posted by Manoj Kasichainula <ma...@io.com>.
On Sat, Oct 09, 1999 at 09:56:43PM +0100, Ben Laurie wrote:
> manoj@hyperreal.org wrote:
> >   --- Configure 1999/10/01 16:22:48     1.8
> >   +++ Configure 1999/10/09 20:28:40     1.9
> >   @@ -1727,7 +1727,7 @@
> >    if [ -d ./lib/apr ]; then
> >        if [ ! -f ./lib/apr/configure ]; then
> >            echo " + bootstrapping Apache Portable Runtime (APR)"
> >   -        (cd lib/apr && autoconf)
> >   +        (cd lib/apr && autoconf && autoheader)
> 
> Ah. This really ought to be triggered by the existence of autoconf, or
> it doesn't actually help much (after the first time), does it?

In my sleepy haze (I work on Hawaiian time on weekends), I can't quite
parse this. Are you saying that we should be checking for the
existence of autoconf first so that error messages are sane?

I agree. I'm not sure whether it's worth the effort to fix up
Configure too much if we're placing the whole ball with autoconf. I'm
probably going to make autoconf my next project, so hopefully this
isn't too far away.

-- 
Manoj Kasichainula - manojk at io dot com - http://www.io.com/~manojk/