From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Make pg_bsd_indent's .h files inclusion-order-safe. |
Date: | 2023-02-12 22:21:41 |
Message-ID: | 20230212222141.hsrabsjgzksfuq77@awork3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Hi,
On 2023-02-12 16:25:28 -0500, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
> > On February 12, 2023 1:11:08 PM PST, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> >> I don't think -Werror helps any though; it's a matter of whether the
> >> cfbot is paying attention to headerscheck's exit status ...
>
> > Thomas fixed that part a while ago, but it'll just return 0 if it just is a warning...
>
> Hmm, but why'd the buildfarm complain then? ... oh, because it checks
> for nonempty output as well as nonzero exit status.
Right.
Obviously that approach could be re-implemented for CI, but givent that it
additionally is somewhat required to make use of headerscheck/cpluspluscheck
locally, it feels better to solve it in headerscheck. And the easiest way for
that seems to be -Werror?
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2023-02-12 22:30:50 | Re: pgsql: Make pg_bsd_indent's .h files inclusion-order-safe. |
Previous Message | Tom Lane | 2023-02-12 21:25:28 | Re: pgsql: Make pg_bsd_indent's .h files inclusion-order-safe. |