Re: ./configure argument checking

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Jim C(dot) Nasby" <jim(at)nasby(dot)net>
Cc: Martijn van Oosterhout <kleptog(at)svana(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Bruce Momjian <bruce(at)momjian(dot)us>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: ./configure argument checking
Date: 2006-10-13 16:45:23
Message-ID: 25469.1160757923@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

"Jim C. Nasby" <jim(at)nasby(dot)net> writes:
> In the meantime, +1 to adding some whitespace around the warning... I'd
> suggest two blank lines before and after.

I don't really see that that would accomplish anything. The problem is
exactly that configure emits many many lines of output which no one
bothers to read --- it's been years since it even fit in my terminal
window's scroll-back buffer :-( A couple blank lines in there won't do
much except make the output even longer.

> ... personally I'd much rather have the default
> be to error-out if it gets garbage arguments. If we provided a flag that
> disabled that, all porters would have to do is to add that to the myriad
> of flags they're already passing in.

This is bending the upstream autoconf developers' idea of what to do
well past the breaking point ;-). Still, if we think that bad configure
arguments is a serious problem, maybe this is what we should do.

The reason I like the "quiet mode" idea better is that "garbage
argument" is not the only warning I fear people are missing. There's
also the one about "you've got an obsolete Bison", which is either
extremely important or utterly useless depending on whether there are
up-to-date prebuilt .c files or not. That means we can *not* turn it
into an error condition ... but because it is not close to either the
beginning or the end of the configure run, it's virtually guaranteed
that people won't notice it. If we fixed things so that the warnings
were pretty nearly the only output, then they'd get noticed.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2006-10-13 16:45:54 Re: [PERFORM] Hints proposal
Previous Message Stephen Frost 2006-10-13 16:44:25 Re: [HACKERS] array_accum aggregate