From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | Sutou Kouhei <kou(at)clear-code(dot)com> |
Cc: | andres(at)anarazel(dot)de, tristan(at)neon(dot)tech, michael(at)paquier(dot)xyz, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: meson: Specify -Wformat as a common warning flag for extensions |
Date: | 2024-06-04 07:00:40 |
Message-ID: | 7f5f3c2c-b61b-4064-b551-aac57d95c879@eisentraut.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 29.05.24 08:47, Sutou Kouhei wrote:
> In <4707d4ed-f268-43c0-b4dd-cdbc7520f508(at)eisentraut(dot)org>
> "Re: meson: Specify -Wformat as a common warning flag for extensions" on Tue, 28 May 2024 23:31:05 -0700,
> Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
>
>> On 07.04.24 18:01, Sutou Kouhei wrote:
>>> +# We don't have "warning_level == 3" and "warning_level ==
>>> +# 'everything'" here because we don't use these warning levels.
>>> +if warning_level == '1'
>>> + common_builtin_flags += ['-Wall']
>>> +elif warning_level == '2'
>>> + common_builtin_flags += ['-Wall', '-Wextra']
>>> +endif
>>
>> I would trim this even further and always export just '-Wall'. The
>> other options aren't really something we support.
>
> OK. How about the v6 patch? It always uses '-Wall'.
Yes, this looks good to me.
All: I think we should backpatch this. Otherwise, meson-based installs
will get suboptimal behavior for extension builds via pgxs.
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2024-06-04 07:18:28 | Re: Proposal: Document ABI Compatibility |
Previous Message | Kyotaro Horiguchi | 2024-06-04 07:00:32 | Re: In-placre persistance change of a relation |