RFR: JDK-8201320 Feature request: Allow PrintFailureReports to be turned off

[Magnus Ihse Bursie]

On 2018-04-10 23:24, Erik Joelsson wrote:
> Hello,
>
> Nice feature!
>
> Init.gmk: 229 were -> was
Fixed without new webrev.

>
> Otherwise looks good.
>
> Out of curiosity, was there a reason to move the log parsing macros 
> outside of has-spec block? It doesn't look like you changed where you 
> call these macros from.
Yes, there was, and yes, I have changed it. :) Right below my typo :-) I 
re-call ParseLogLevel if I get a value in DEFAULT_LOG from the spec.gmk. 
Unfortunately, this means that I now need to call ParseLogLevel both 
without a spec and with a spec, which I have hitherto treated as 
completely different scenarios in Init.gmk/InitSupport.gmk. I have 
verified that the code is suitable to run in the new situation of being 
with a spec.gmk as well. There's a fix for COMMA that's not needed when 
running with a spec, but it doesn't harm either so it's okay.

/Magnus

>
> /Erik
>
>
> On 2018-04-10 13:54, Magnus Ihse Bursie wrote:
>> From the bug report:
>>
>> "The compile errors you get from HotSpot are quite large, and usually 
>> don't get entirely printed in PrintFailureReports. This has the 
>> effect that the goto mode to find the compilation error is to scroll 
>> past PrintFailureReports to get to the complete error message.
>>
>> It would be nice if there was a way to turn off this feature from the 
>> command line."
>>
>> I've solved this by adding a new LOG option, "report", which takes an 
>> argument: "report=default", "report=none" or "report=all". As usual, 
>> this can be combined with other LOG options, e.g. "LOG=info,report=all".
>>
>> The "default" value is what it always been, giving you the first 
>> screenful of lines of each failure. "none" is what Stefan requested, 
>> and "all" means that there is no truncating, so in a sense, it's 
>> another way of giving Stefan what he wants. :-)
>>
>> To make this usable in practice, I also implemented a feature I've 
>> been thinking about a long time, but never gotten around to. And that 
>> is to be able to set a default value for LOG in configure, similar to 
>> how we can set default values for JOBS or the default make target.
>>
>> The new flag is "--with-log=<LOG value>", e.g. 
>> "--with-log=info,report=none". If a LOG= value is given on the 
>> command line, it overrides the default value provided to configure.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8201320
>> WebRev: 
>> http://cr.openjdk.java.net/~ihse/JDK-8201320-allow-disabling-of-exit-reports/webrev.01
>>
>>
>> /Magnus
>>
>