Re: [PATCH] Makefile: make COMPUTE_HEADER_DEPENDENCIES=auto work with DEVOPTS=pedantic

From: Carlo Arenas <carenas@gmail.com>
Date: Sep 22, 2021, 10:55 UTC
Message-ID: <CAPUEsph8TYokqLmiZjvu3VgmzLh4AFhp+0vh0nv+2+m351=n_A@mail.gmail.com>
In-Reply-To: <patch-1.1-6b2e9af5e67-20210922T103749Z-avarab@gmail.com>

On Wed, Sep 22, 2021 at 3:38 AM Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:

Show 5 quoted lines

>
> The "COMPUTE_HEADER_DEPENDENCIES" feature added in [1] was extended to
> use auto-detection in [2]. Then when -Wpedantic support was added to
> DEVOPTS in [3] we started passing -Wpedantic in combination with
> -Werror to the compiler here.

It is my impression though that header dependencies computation is something that might only work in gcc and clang (because of its gcc compatibility), so shouldn't this be restricted to those compilers instead of forcing all others to error?

Also, why is this process run with DEVELOPER=1 to begin with, if we obviously don't need/want any compilations warnings?

Carlo