RFR: 8146132: Excessive output from make test-image [v2]

Magnus Ihse Bursie ihse at openjdk.org
Fri Jan 13 04:03:01 UTC 2023


On Thu, 12 Jan 2023 18:01:50 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> This change makes native output files be only printed when building with LOG=info, to make the build not have so many pages of output.
>> Tested with GHA and locally (tbd)
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   This seems to do the right thing without summarizing the number of object files compiled for the tests.

Hm. I fully agree on the problem description, but not entirely sure about this solution. In contrast with Erik, I was more thinking along the lines that the decision to be more quiet could be made by SetupNativeCompilation itself, based on the number of files it compiled. So if it only compiled a single c file into a single dll, then it should be quiet.

With "meta message", I think Erik means that if all tests are suddenly being quiet, we should compensate by printing a more general high-level message like 
"Building NN native tests".

-------------

PR: https://git.openjdk.org/jdk/pull/11930



More information about the build-dev mailing list