Re: JDK 9 RFR of JDK-8177638: com/sun/jarsigner, jdk/internal/loader (and more) are missed in TEST.group

2017-03-28 Thread Amy Lu
Ping ~ On 3/27/17 1:12 PM, Amy Lu wrote: Sure. Moved to jdk_security3. webrev updated: http://cr.openjdk.java.net/~amlu/8177638/webrev.00/ Thanks, Amy On 3/27/17 11:25 AM, Weijun Wang wrote: The com/sun/jarsigner line should be added to jdk_security3. Roughly, the categories mean: 1: publi

Re: RFR of 8177683: Suppress lint removal warnings in jdk.security and jdk.policytool

2017-03-28 Thread Weijun Wang
But the "deprecation" argument is also for PolicyTool. -Max On 03/29/2017 07:35 AM, joe darcy wrote: Hi Max, Since entire classes were being covered by the annotation, I wanted to make clear that the removal warnings in those cases were for PolicyTool as opposed to being for something else. T

Re: RFR of 8177683: Suppress lint removal warnings in jdk.security and jdk.policytool

2017-03-28 Thread joe darcy
Hi Max, Since entire classes were being covered by the annotation, I wanted to make clear that the removal warnings in those cases were for PolicyTool as opposed to being for something else. Thanks, -Joe On 3/28/2017 4:30 PM, Weijun Wang wrote: Hi Joe This looks fine to me. One question

Re: RFR of 8177683: Suppress lint removal warnings in jdk.security and jdk.policytool

2017-03-28 Thread Weijun Wang
Hi Joe This looks fine to me. One question: -@SuppressWarnings("deprecation") +@SuppressWarnings({"deprecation", + "removal"}) // PolicyTool Why did you split the line to 2? Thanks Max On 03/28/2017 09:09 AM, joe darcy wrote: Hello, The jdk.security and jdk.policytool mod

Re: [PATCH] 8005819: Support cross-realm MSSFU

2017-03-28 Thread Weijun Wang
Hi Richard I'll look into it. I used to think Kerberos referral is a must to support cross-realm MSSFU but I could be wrong. What platforms have you tested this on? MSAD? MIT krb5? Two reminders: 1. We don't have time for JDK 9 now. So this can only go into JDK 10. 2. Have you signed the Or

[PATCH] 8005819: Support cross-realm MSSFU

2017-03-28 Thread richard
Hi, I have developed a fix for the lack of cross-realm S4USelf support in OpenJDK 8 which I would like to contribute. The fix does not address realm referral as discussed in the bug and so requires both realms to be present in the krb5.conf file. Bug: https://bugs.openjdk.java.net/browse/J