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

From: Sandeep Thakkar <sandeep(dot)thakkar(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, 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-06-21 04:32:44
Message-ID: CANFyU95MMxUDF7k4TqhPLMQ9KDyZeET26ocpvmbr8tzawrC1nQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

Do we see any solution to this issue? or using the older SDK is the way to
go?

On Thu, May 20, 2021 at 2:04 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
> On Tue, Mar 30, 2021 at 6:58 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
>> Thomas Munro <thomas(dot)munro(at)gmail(dot)com> writes:
>> > I'll move it when committing. I'll let this patch sit for another day
>> > to see if any other objections show up.
>>
>> FWIW, I remain fairly strongly against this, precisely because of the
>> point that it requires us to start using a randomly different
>> feature-probing technology anytime Apple decides that they're going to
>> implement some standard API that they didn't before. Even if it works
>> everywhere for preadv/pwritev (which we won't know in advance of
>> buildfarm testing, and maybe not then, since detection failures will
>> probably be silent), it seems likely that we'll hit some case in the
>> future where this interacts badly with some other platform's weirdness.
>> We haven't claimed in the past to support MACOSX_DEPLOYMENT_TARGET,
>> and I'm not sure we should start now. How many people actually care
>> about that?
>>
>
> I missed this earlier - it's come to my attention through a thread on the
> -packagers list. Adding my response on that thread here for this audience:
>
> The ability to target older releases with a newer SDK is essential for
> packages such as the EDB PostgreSQL installers and the pgAdmin community
> installers. It's very difficult (sometimes impossible) to get older OS
> versions on new machines now - Apple make it very hard to download old
> versions of macOS (some can be found, others not), and they won't always
> work on newer hardware anyway so it's really not feasible to have all the
> build machines running the oldest version that needs to be supported.
>
> FYI, the pgAdmin and PG installer buildfarms have
> -mmacosx-version-min=10.12 in CFLAGS etc. to handle this, which is
> synonymous with MACOSX_DEPLOYMENT_TARGET. We've been successfully building
> packages that way for a decade or more.
>
> --
> Dave Page
> Blog: https://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EDB: https://www.enterprisedb.com
>
>

--
Sandeep Thakkar

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Mark Dilger 2021-06-21 04:54:53 Re: Optionally automatically disable logical replication subscriptions on error
Previous Message Peter Geoghegan 2021-06-21 04:28:15 Re: Maintaining a list of pgindent commits for "git blame" to ignore