Re: [PATCH v3 1/1] Fix detection of preadv/pwritev support for OSX.

From: Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>
To: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Sandeep Thakkar <sandeep(dot)thakkar(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, James Hilliard <james(dot)hilliard1(at)gmail(dot)com>, David Steele <david(at)pgmasters(dot)net>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Sergey Shinderuk <s(dot)shinderuk(at)postgrespro(dot)ru>
Subject: Re: [PATCH v3 1/1] Fix detection of preadv/pwritev support for OSX.
Date: 2021-07-03 11:38:53
Message-ID: 6ac601c7-9a72-d354-cfaa-03a66108c778@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 21.06.21 07:22, Thomas Munro wrote:
> I'm not personally against the proposed change. I'll admit there is
> something annoying about Apple's environment working in a way that
> doesn't suit traditional configure macros that have been the basis of
> portable software for a few decades, but when all's said and done,
> configure is a Unix wars era way to make things work across all the
> Unixes, and most of them are long gone, configure itself is on the way
> out, and Apple's still here, so...

I think this change is perfectly appropriate (modulo some small cleanups).

The objection was that you cannot reliably use AC_CHECK_FUNCS (and
therefore AC_REPLACE_FUNCS) anymore, but that has always been true,
since AC_CHECK_FUNCS doesn't handle macros, compiler built-ins, and
functions that are not declared, and any other situation where looking
for a symbol in a library is not the same as checking whether the symbol
actual works for your purpose. This is not too different from the long
transition from "does this header file exists" to "can I compile this
header file".

So in fact the correct way forward would be to get rid of all uses of
AC_CHECK_FUNCS and related, and then this problem goes away by itself.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andy Fan 2021-07-03 14:08:26 Re: Keep notnullattrs in RelOptInfo (Was part of UniqueKey patch series)
Previous Message Peter Eisentraut 2021-07-03 10:44:15 Re: Remove useless int64 range checks on BIGINT sequence MINVALUE/MAXVALUE values