Re: Change copyObject() to use typeof_unqual

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: Jelte Fennema-Nio <postgres(at)jeltef(dot)nl>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Change copyObject() to use typeof_unqual
Date: 2026-03-13 13:03:20
Message-ID: 68C5A719-919A-483D-96F9-7D59CEA6D519@yesql.se
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 13 Mar 2026, at 11:43, Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:

> I committed this and it still fails, but the failure is now narrower. There is a failure on buildfarm member taipan because it uses an unusual combination of gcc and clang (the gcc is much newer than clang). The only sensible workaround I could think of is a hardcoded override based on the clang version, as in the attached patch. And alternative is that we decide that we don't want to support this combination, meaning that we would effectively require that clang is approximately as-old-or-newer than gcc.

I ran into this as well on clang 15 via XCode with no gcc involved:

../src/test/modules/test_cplusplusext/test_cplusplusext.cpp:41:22: error: no template named 'remove_reference_t' in namespace 'std'; did you mean 'remove_reference'?
RangeTblRef *copy = copyObject(nodec);
^~~~~~~~~~~~~~~~~
--
Daniel Gustafsson

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alena Rybakina 2026-03-13 13:04:24 Re: Vacuum statistics
Previous Message Alena Rybakina 2026-03-13 12:51:17 Re: Vacuum statistics