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

2018-01-12 Thread Roger Riggs
ok, in that case, no additional Warning message is needed, the useful warning has already been printed. Thanks, Roger On 1/12/2018 10:42 AM, Kumar Srinivasan wrote: Hi Roger, Thanks for looking at this. Hi Kumar, RunPathTest should say *what* is missing from the environment to skip the te

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

2018-01-12 Thread Kumar Srinivasan
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 messag

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

2018-01-12 Thread Roger Riggs
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. 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  bas

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

2018-01-11 Thread Kumar Srinivasan
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