Re-ran passed.
Webrev updated at: http://cr.openjdk.java.net/~valeriep/8154191/jdk/webrev.01/
Regards,
Valerie

On 6/14/2016 4:05 PM, Valerie Peng wrote:
That sounds better. I will update the test and re-run it.
Thanks,
Valerie

On 6/14/2016 2:28 PM, Sean Mullan wrote:
I don't think you need to make changes to test/javax/smartcardio/policy. Instead you can change the tests that use this policy file to use the new jtreg @run /java.security.policy=policy which extends the default system policy.

--Sean


On 06/13/2016 08:10 PM, Valerie Peng wrote:
Sean,

Can you please review the changes for deprivileging the java.smartcardio
module?
I have to update one makefile in the top-level workspace besides the
java.policy file in the jdk workspace.
One java.smartcardio regression test uses custom policy file which needs
to be updated as well.

Bug: https://bugs.openjdk.java.net/browse/JDK-8154191
Webrevs:
top-level=> http://cr.openjdk.java.net/~valeriep/8154191/webrev.00/
jdk=> http://cr.openjdk.java.net/~valeriep/8154191/jdk/webrev.00/

Thanks,
Valerie

Reply via email to