JDK-8150990: [packager] Research Refactoring Packager

[Kevin Rushforth]

Looks good.

-- Kevin


Chris Bensen wrote:
> 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
>>>>         
>
>