Re: RFR 8209995: java.base does not need to export sun.security.ssl to java.security.jgss

2018-08-27 Thread Bradford Wetmore
> Looks good. +1. Brad On 8/27/2018 8:12 AM, Alan Bateman wrote: On 27/08/2018 16:09, Weijun Wang wrote: Since we've removed all krb5-based ciphersuites from JSSE, this qualified export is useless anymore. Please review the patch below: diff --git a/src/java.base/share/classes/module-in

Re: RFR 8209995: java.base does not need to export sun.security.ssl to java.security.jgss

2018-08-27 Thread Alan Bateman
On 27/08/2018 16:09, Weijun Wang wrote: Since we've removed all krb5-based ciphersuites from JSSE, this qualified export is useless anymore. Please review the patch below: diff --git a/src/java.base/share/classes/module-info.java b/src/java.base/share/classes/module-info.java --- a/src/java.b

Re: RFR 8209995: java.base does not need to export sun.security.ssl to java.security.jgss

2018-08-27 Thread Xuelei Fan
Looks fine to me. Thanks, Xuelei On 8/27/2018 8:09 AM, Weijun Wang wrote: Since we've removed all krb5-based ciphersuites from JSSE, this qualified export is useless anymore. Please review the patch below: diff --git a/src/java.base/share/classes/module-info.java b/src/java.base/share/class

RFR 8209995: java.base does not need to export sun.security.ssl to java.security.jgss

2018-08-27 Thread Weijun Wang
Since we've removed all krb5-based ciphersuites from JSSE, this qualified export is useless anymore. Please review the patch below: diff --git a/src/java.base/share/classes/module-info.java b/src/java.base/share/classes/module-info.java --- a/src/java.base/share/classes/module-info.java +++ b/s