Re: [PATCH] Fix OAuth feature detection on OpenBSD+Meson - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH] Fix OAuth feature detection on OpenBSD+Meson
Date
Msg-id 1172336.1750803471@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH] Fix OAuth feature detection on OpenBSD+Meson  (Jacob Champion <jacob.champion@enterprisedb.com>)
List pgsql-hackers
Jacob Champion <jacob.champion@enterprisedb.com> writes:
> On Tue, Jun 24, 2025 at 2:50 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Can't say that I find this to be impressive software engineering:
>> rather than having only one probe failure mode to worry about,
>> we have two, depending on whether the compiler knows __has_include().
>> Pretty close to the worst of all possible worlds.

> I did a double-take on the code you posted, but! It looks like they're
> running the preprocessor only. That doesn't seem so bad to me (though
> they could probably do better than calling it a "compile" in the log).

Oh!  Okay, then the two cases should be mostly semantically
equivalent, I think, ie it's just a "does the header exist" test.
There are probably some edge cases where the effects are different,
but nothing that would be likely to matter for us.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Jacob Champion
Date:
Subject: Re: [PATCH] Fix OAuth feature detection on OpenBSD+Meson
Next
From: Tom Lane
Date:
Subject: Re: Add \pset options for boolean value display