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