Ugh, day of mistakes. Here’s the correct link: http://cr.openjdk.java.net/~cbensen/JDK-8150990/webrev.02/
> On Apr 28, 2016, at 7:07 AM, Chris Bensen <chris.ben...@oracle.com> wrote: > > Doesn’t look like it’s been pushed. I revised the change: > > http://cr.openjdk.java.net/~cbensen/JDK-8150990/webrev.01/ > > Chris > > >> On Apr 28, 2016, at 6:55 AM, Chris Bensen <chris.ben...@oracle.com> wrote: >> >> If it hasn’t been pushed I’ll change it? >> >> Chris >> >> >>> On Apr 27, 2016, at 11:58 PM, Alan Bateman <alan.bate...@oracle.com> wrote: >>> >>> >>> >>> On 27/04/2016 21:44, Kevin Rushforth wrote: >>>> Looks fine to me, too. >>>> >>>> Btw, I thought Mandy had earlier suggested >>>> jdk.tools.jlink.internal.packager as a name, but it's currently >>>> jdk.tools.internal.packager (without the jlink). I don't care one way or >>>> the other and the current one is shorter. >>> Mandy's suggestion sounds right to me as jlink already has internals in >>> jdk.tools.jlink.internal.**. If it's temporary then the suggest is probably >>> okay although if temporary means to JDK 9 GA then it might be better to >>> change it now. >>> >>> -Alan >> >
