Feedback-ID: ie0a040ee:Fastmail
Message-ID: <5a256041-2520-4082-8483-7f8c1e79fb88@eisentraut.org>
Date: Wed, 15 May 2024 07:20:09 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: Requiring LLVM 14+ in PostgreSQL 18
To: Thomas Munro <thomas.munro@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
References: <CA+hUKG+SOP-aR=YF_n0dtXGWeCy6x+Cn-RMWURU5ySQdmeKW1Q@mail.gmail.com>
 <CA+hUKGLXr0Rjz_6PQrNWe7Z-k7_G-MAHoh1DP+A-sxw0qpYEDQ@mail.gmail.com>
 <CA+hUKGLFNFMOPPME6mpkGeve0+6CRrZLMTKs+MmBGJp4vLcJ=A@mail.gmail.com>
 <5255e75e-5a66-41ac-99d3-1998283b86e6@eisentraut.org>
 <CA+hUKGL3EJ37xuj0813AfgwtxBSmfM_pwE8pSGGE5BeKSxCiZw@mail.gmail.com>
Content-Language: en-US
From: Peter Eisentraut <peter@eisentraut.org>
In-Reply-To: <CA+hUKGL3EJ37xuj0813AfgwtxBSmfM_pwE8pSGGE5BeKSxCiZw@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Archived-At: <https://www.postgresql.org/message-id/5a256041-2520-4082-8483-7f8c1e79fb88%40eisentraut.org>
Precedence: bulk

On 15.05.24 06:21, Thomas Munro wrote:
>> And as I'm looking up how this was previously handled, I notice that
>> this list of clang-NN versions was last updated equally sneakily as part
>> of your patch to trim off LLVM <10 (820b5af73dc).  I wonder if the
>> original intention of that configure code was that maintaining the
>> versioned list above clang-7/llvm-config-7 was not needed, because the
>> unversioning programs could be used, or maybe because pkg-config could
>> be used.  It would be nice if we could get rid of having to update that.
> I probably misunderstood why we were doing that, perhaps something to
> do with the way some distro (Debian?) was doing things with older
> versions, and yeah I see that we went a long time after 7 without
> touching it and nobody cared.  Yeah, it would be nice to get rid of
> it.  Here's a patch.  Meson didn't have that.

Yes, let's get that v3-0001 patch into PG17.