Re: CompilerWarnings task does not catch C++ warnings - Mailing list pgsql-hackers

From Andres Freund
Subject Re: CompilerWarnings task does not catch C++ warnings
Date
Msg-id 20240618143116.sgjwk3aynyv5wlw3@awork3.anarazel.de
Whole thread Raw
In response to CompilerWarnings task does not catch C++ warnings  (Peter Eisentraut <peter@eisentraut.org>)
List pgsql-hackers
Hi,

On 2024-06-18 09:27:02 +0200, Peter Eisentraut wrote:
> The CompilerWarnings task on Cirrus CI does not catch warnings in C++ code.
> It tries to make warnings fatal by passing COPT='-Werror', but that does not
> apply to C++ compilations.
> 
> I suggest that we just add COPT to CXXFLAGS as well.  I think passing
> -Werror is just about the only reasonable use of COPT nowadays, so making
> that more robust seems useful.  I don't think there is a need for a separate
> make variable for C++ here.

+1

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Yura Sokolov
Date:
Subject: Re: Suggest two small improvements for PITR.
Next
From: Andres Freund
Date:
Subject: Re: Truncation of mapped catalogs (whether local or shared) leads to server crash