Re: cpluspluscheck vs ICU again - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: cpluspluscheck vs ICU again
Date
Msg-id 0a6683d6-e7ac-4957-bfad-eff06f115e2b@eisentraut.org
Whole thread Raw
In response to implicit casts from void*  (John Naylor <johncnaylorls@gmail.com>)
List pgsql-hackers
On 02.07.25 09:01, John Naylor wrote:
> On Tue, Jul 1, 2025 at 9:24 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Ha, indeed you are right.  On my RHEL9 box, it's kinda drowned out
>> by complaints about
>>
>> /usr/include/c++/11/bits/range_access.h:109:3: error: template with C linkage
>>    109 |   template<typename _Tp> _Tp* end(valarray<_Tp>&) noexcept;
>>        |   ^~~~~~~~
>> /tmp/headerscheck.u5CrRM/test.cpp:1:1: note: ‘extern "C"’ linkage started here
>>      1 | extern "C" {
>>        | ^~~~~~~~~~
> 
> After pushing my fix, I looked into this, and CI works around this by
> disabling ICU. A proper fix was discussed here, but it trailed off:
> 
>
https://www.postgresql.org/message-id/flat/20230311033727.koa4saxy5wyquu6s%40awork3.anarazel.de#03346c63050bbc69dfca8981a5698e4a
> 
> I came up with the attached -- Andres, Peter, does this match your recollection?

This looks sensible to me.  Assuming that it works for this purpose, it 
seems otherwise harmless.




pgsql-hackers by date:

Previous
From: shveta malik
Date:
Subject: Re: Using failover slots for PG-non_PG logical replication
Next
From: Nazir Bilal Yavuz
Date:
Subject: Explicitly enable meson features in CI