On Tue, Jul 28, 2020 at 10:33:52 +0200, Pavel Hrdina wrote:
> On Tue, Jul 28, 2020 at 10:09:42AM +0200, Peter Krempa wrote:

[...]

> 
> Here I disagree, it also means that compilation should not fail. We do
> comments all the time to patch series that every single commit should
> compile correctly on it's own within the series.

Doing a 'return 0' is not compiling code correctly. The idea of
requirements to bulild cleanly is that you can test the code.

In this case we need to observe it from a different angle though.

In reality an incomplete build is a failed build regardless of what the
return value of the build system is. And that is important here. The
main reason for the build system is to build everything so the only
success is when everything is built.

If you don't have the resulting binary it's impossible to test the code
or do anything else.

What would be even worse is to get a compiled binary that e.g. doesn't
have the dependencies installed (such as RNG schemas, cpu XML docs and
such) and fails in magic ways. At that point you can't be sure whether
it's the bug you are trying to locate or just plainly broken build
which the build system lied to you that it's complete.

Reply via email to