Re: new buildfarm with gcc & clang "trunk" -Werror?

From: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: new buildfarm with gcc & clang "trunk" -Werror?
Date: 2018-03-29 13:52:29
Message-ID: alpine.DEB.2.20.1803291546360.11245@lancre
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Hello Peter,

>> - fix these warnings (other than putting -Wno-format-truncation or
>> similar workarounds...).
>
> I've been tracking gcc-8, and I have a patch ready, but these things
> change a bit with every compiler snapshot, so I'm waiting until things
> settle down.

Ok, so I will not submit anything for gcc. I'll look at clang, though.

>> - add permanent gcc/clang trunk beasts with -Werror
>> (if so, I'd suggest cannonball & seanettle for the names).
>
> I would not like that. The build farm success should ideally be a
> function of correct PostgreSQL code, not external factors. If an
> in-progress compiler does funny things, what are we supposed to do about
> that?

That happens infrequently, commits in gcc & clang are supposed not to
break the compiler, just like with postgres:-) When it happens (3 times in
6 months), I just report a bug on the compiler side, which is usually
fixed in under one week.

> Generally, fixing up PostgreSQL for a new compiler release isn't a major
> effort and can be done briefly before the release of the compiler.

Sure.

So I understand that a -Werror beast is somehow not desirable.

--
Fabien.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2018-03-29 13:55:50 Re: Parallel Aggregates for string_agg and array_agg
Previous Message Stephen Frost 2018-03-29 13:51:10 Re: pgsql: Add documentation for the JIT feature.