Re: [HACKERS] 6.4 BETA2 fails to compile on Digital Unix 4.0d - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [HACKERS] 6.4 BETA2 fails to compile on Digital Unix 4.0d
Date
Msg-id 5013.909683149@sss.pgh.pa.us
Whole thread Raw
In response to Re: [HACKERS] 6.4 BETA2 fails to compile on Digital Unix 4.0d  ("Pedro J. Lobo" <pjlobo@euitt.upm.es>)
List pgsql-hackers
"Pedro J. Lobo" <pjlobo@euitt.upm.es> writes:
> Ok, here's what happens: the line "AC_PROG_GCC_TRADITIONAL" in
> configure.in expands to the following in configure:

> if test $ac_cv_prog_gcc = yes; then

> The problem is that $ac_cv_prog_gcc is empty if you specify
> "--with-CC=", so the first line expands to "if test  = yes; then", which
> is clearly incorrect.

Ah.  OK, Autoconf is maybe not being too robust here, but it's our bug.
AC_PROG_GCC_TRADITIONAL assumes you've run AC_PROG_CC, and we don't do
that if --with-CC is given.

I will rearrange the configure script so that that's true.  As far as I
can see, there's no reason to run AC_PROG_GCC_TRADITIONAL at all if the
user specifies --with-CC ... if the user wants to override the script's
idea of what C compiler to use, then it's also the user's responsibility
to supply any special CFLAGS that might be needed.  So we can just move
the macro into the "else" case where AC_PROG_CC is run.

The -traditional switch shouldn't be needed anyway in any reasonably
modern gcc installation, so we could probably get away with just
removing that test entirely.  But I'll leave it in.

> I don't know if I will be able to test BETA4, because I'm leaving for a
> short vacation (lucky me :-) on Saturday and won't be back until Thursday.

Have a good trip!
        regards, tom lane


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] Open 6.4 items
Next
From: "Thomas G. Lockhart"
Date:
Subject: Re: [HACKERS] Open 6.4 items