[RFR] [8u] 8220641, , New test KdcPolicy.java introduced by JDK-8164656 needs same change as JDK-8190690

2019-03-14 Thread Andrew John Hughes
Bug: https://bugs.openjdk.java.net/browse/JDK-8220641 Webrev: https://cr.openjdk.java.net/~andrew/openjdk8/8220641/webrev.01/ This is the patch we split out from my original post for 8175120. It applies the same change to the @run command in KdcPolicy.java as was applied to the tests deleted by 81

Re: [RFR] [8u] 8220641, , New test KdcPolicy.java introduced by JDK-8164656 needs same change as JDK-8190690

2019-03-15 Thread Severin Gehwolf
Hi Andrew, On Fri, 2019-03-15 at 04:55 +, Andrew John Hughes wrote: > Bug: https://bugs.openjdk.java.net/browse/JDK-8220641 > Webrev: > https://cr.openjdk.java.net/~andrew/openjdk8/8220641/webrev.01/ > > This is the patch we split out from my original post for 8175120. It > applies the same

Re: [RFR] [8u] 8220641, , New test KdcPolicy.java introduced by JDK-8164656 needs same change as JDK-8190690

2019-03-15 Thread Hohensee, Paul
+1 Paul On 3/15/19, 4:00 AM, "jdk8u-dev on behalf of Aleksey Shipilev" wrote: On 3/15/19 5:55 AM, Andrew John Hughes wrote: > Bug: https://bugs.openjdk.java.net/browse/JDK-8220641 > Webrev: https://cr.openjdk.java.net/~andrew/openjdk8/8220641/webrev.01/ Looks good, ship i

Re: [RFR] [8u] 8220641, , New test KdcPolicy.java introduced by JDK-8164656 needs same change as JDK-8190690

2019-03-15 Thread Aleksey Shipilev
On 3/15/19 5:55 AM, Andrew John Hughes wrote: > Bug: https://bugs.openjdk.java.net/browse/JDK-8220641 > Webrev: https://cr.openjdk.java.net/~andrew/openjdk8/8220641/webrev.01/ Looks good, ship it. -Aleksey signature.asc Description: OpenPGP digital signature

Re: [RFR] [8u] 8220641, , New test KdcPolicy.java introduced by JDK-8164656 needs same change as JDK-8190690

2019-03-15 Thread Andrew John Hughes
On 15/03/2019 17:58, Hohensee, Paul wrote: > +1 > > Paul > > On 3/15/19, 4:00 AM, "jdk8u-dev on behalf of Aleksey Shipilev" > wrote: > > On 3/15/19 5:55 AM, Andrew John Hughes wrote: > > Bug: https://bugs.openjdk.java.net/browse/JDK-8220641 > > Webrev: https://cr.openjdk.java.net/