On 2017-11-27 22:52, Erik Joelsson wrote:
You need a definition of the variable? Couldn't the user just be required to define it when running tests?
I'm not sure what you mean. The intention is to not only print the output, but to also store it in the test-results directory. I don't think the test result directory should be specifically user overridable. If it should be (why?), there's more places to update.

/Magnus

Looks good.

/Erik


On 2017-11-27 11:59, Magnus Ihse Bursie wrote:
On 2017-11-27 20:02, Erik Joelsson wrote:
Looks good.
Only it didn't. :-)

I managed to drop one (very important) line when juggling this patch between my sandbox and the repo.

Updated WebRev: http://cr.openjdk.java.net/~ihse/JDK-8191923-save-run-test-summary-to-file/webrev.02

/Magnus


/Erik


On 2017-11-27 10:41, Magnus Ihse Bursie wrote:
We should save the run-test summary to a file.

Bug: https://bugs.openjdk.java.net/browse/JDK-8191923
WebRev: http://cr.openjdk.java.net/~ihse/JDK-8191923-save-run-test-summary-to-file/webrev.01

/Magnus




Reply via email to