On 5/3/2016 9:42 AM, Mandy Chung wrote:
On May 3, 2016, at 2:37 AM, Frank Yuan <frank.y...@oracle.com> wrote:
Hi Mandy and Joe
I have re-fixed the bug with your comment, would you like to check again?
http://cr.openjdk.java.net/~fyuan/8155514/webrev.01/
This patch extending the default security policy is a better fix. Looks fine.
Looks good to me too.
I saw that you already pushed the previous version to jdk9/dev. Please create
a new JBS issue for this patch since you can’t push with a duplicated bug ID.
Indeed. And also, the Bug Id in the Subject is incorrect, that's
somebody else's enhancement.
-Joe
Mandy
Thanks,
Frank
-----Original Message-----
From: Mandy Chung [mailto:mandy.ch...@oracle.com]
Sent: Friday, April 29, 2016 4:00 AM
To: Frank Yuan <frank.y...@oracle.com>
Cc: core-libs-dev <core-libs-dev@openjdk.java.net>; huizhe wang
<huizhe.w...@oracle.com>; Xueming Shen
<xueming.s...@oracle.com>
Subject: Re: RFR: 8155600: jaxp.library.TestPolicy should extend the default
security policy
Hi Frank,
The fix looks fine.
Thanks for verifying it with my patch to deprivilege jdk.charasets.
Mandy
On Apr 28, 2016, at 1:11 AM, Frank Yuan <frank.y...@oracle.com> wrote:
Hi Mandy, Joe and all
Would you like to review the fix for bug
https://bugs.openjdk.java.net/browse/JDK-8155514?
The webrev is at: http://cr.openjdk.java.net/~fyuan/8155514/webrev.00/.
Thanks,
Frank
