public inbox for [email protected]
help / color / mirror / Atom feedFrom: Tom Lane <[email protected]>
To: Japin Li <[email protected]>
Cc: Andres Freund <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Cannot find a working 64-bit integer type on Illumos
Date: Fri, 22 Mar 2024 13:25:56 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <ME3P282MB3166D876E9CB83009727A5ECB6312@ME3P282MB3166.AUSP282.PROD.OUTLOOK.COM>
References: <ME3P282MB3166F9D1F71F787929C0C7E7B6312@ME3P282MB3166.AUSP282.PROD.OUTLOOK.COM>
<[email protected]>
<ME3P282MB3166D876E9CB83009727A5ECB6312@ME3P282MB3166.AUSP282.PROD.OUTLOOK.COM>
Japin Li <[email protected]> writes:
> On Sat, 23 Mar 2024 at 00:53, Andres Freund <[email protected]> wrote:
>> Likely there's an unrelated warning triggering the configure test to
>> fail. We'd need to see config.log to see what that is.
> Thanks for your quick reply. Attach the config.log.
Yup:
conftest.c:139:5: error: no previous prototype for 'does_int64_work' [-Werror=missing-prototypes]
139 | int does_int64_work()
| ^~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
configure:17003: $? = 1
configure: program exited with status 1
This warning is harmless normally, but breaks the configure probe if
you enable -Werror.
No doubt we could improve that test snippet so that it does not
trigger that warning. But trying to make configure safe for -Werror
seems like a fool's errand, for these reasons:
* Do you really want to try to make all of configure's probes proof
against every compiler warning everywhere?
* Many of the test snippets aren't readily under our control, as they
are supplied by Autoconf.
* In the majority of cases, any such failures would be silent, as
configure would just conclude that the feature it is probing for
isn't there. So even finding there's a problem would be difficult.
The short answer is that Autoconf is not designed to support -Werror
and it's not worth it to try to make it do so.
regards, tom lane
view thread (6+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: Cannot find a working 64-bit integer type on Illumos
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox