On 2017-11-27 14:19, Magnus Ihse Bursie wrote:
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?
Sorry, I meant this as a joke but forgot to add the smiley. :)

/Erik
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