On 10/28/2011 10:05 PM, Weijun Wang wrote: > Oh, really? On my Linux it's always ignored. > Yes, it's true. If I'm correct, I also address similar issues on other CRs.
> What did you mean "When run the test manually"? javac and java it? > By remove @ignore tag, and jtreg. > Anyway, the fix is correct. You can putback it. > Thanks! Xuelei > Thanks > Max > > On 10/28/2011 09:55 PM, Xuelei Fan wrote: >> Move forward the @run tag, so that it can be really ignored by @ignore >> tag. Otherwise, it will be run as normal. That's also why I can catch >> the exception. >> >> Thanks, >> Xuelei >> >> On 10/28/2011 9:53 PM, Weijun Wang wrote: >>> CipherTest looks fine. What is the change in ClientJSSEServerJSSE for? >>> Maybe you meant to remove the @ignore line? >>> >>> Thanks >>> Weijun >>> >>> >>> On 10/28/2011 09:19 PM, Xuelei Fan wrote: >>>> Hi Weijun, >>>> >>>> Would you please review my fix for 7105940(Test regression: KeyStore >>>> must be from provider SunPKCS11-NSSKeyStore)? >>>> >>>> webrev: http://cr.openjdk.java.net/~xuelei/7105940/webrev.00/ >>>> >>>> Thanks, >>>> Xuelei >>