RFR: JDK-8191923 Save run-test summary to file

[Erik Joelsson]

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