The change looks great. Thank you for removing another shell test. --Max
> On Jul 5, 2016, at 1:56 AM, Sibabrata Sahoo <sibabrata.sa...@oracle.com> > wrote: > > Hi, > > Please review this patch for “8160624: sun/security/tools/keytool/printssl.sh > failed with "Socket closed" > > JBS: https://bugs.openjdk.java.net/browse/JDK-8160624 > Webrev: http://cr.openjdk.java.net/~ssahoo/8160624/webrev.00/ > > Description: > The test is for “keytool should be able to import certificates from remote > SSL server”. The failure was due to the serversocket which get pre-closed > during communication. This patch simplify the test by another approach of > implementation and fix such issues to not appear in future. > > Thanks, > Siba