JDK-8150990: [packager] Research Refactoring Packager

[Chris Bensen]

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.bensen at 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.bensen at 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.Bateman at 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
>> 
>