Looks fine.
--Sean
On 06/14/2016 08:12 PM, Valerie Peng wrote:
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
