RFR: 8194287 and 8194286: simple test changes for the execution of these tests in a docket container

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Fri Jan 12 15:42:00 UTC 2018


Hi Roger,

Thanks for looking at this.

> Hi Kumar,
>
> RunPathTest should say *what* is missing from the environment to skip 
> the test.
> Otherwise, the curious has to go back a read the code to figure it out.


Actually it  *does* say *what* is missing, however it will be printed 
before this message.

webrev does not quite make this clear as it gobbles up areas of the 
source files which have
are not changed, please see this line, it will become apparent.

http://cr.openjdk.java.net/~ksrini/8194287/webrev-00/test/jdk/tools/launcher/RunpathTest.java.html 

line 52

Thanks
Kumar
>
> Thanks, Roger
>
>
> On 1/11/2018 6:00 PM, Kumar Srinivasan wrote:
>> Hi,
>>
>> Please review bug fixes for 2 test bugs, which fails with a base 
>> Linux installation
>> in a docker container.
>>
>> Thanks
>> Kumar
>>
>> [1] http://cr.openjdk.java.net/~ksrini/8194287/webrev-00/
>> [2] http://cr.openjdk.java.net/~ksrini/8194287/cmt.txt
>



More information about the core-libs-dev mailing list