RFR: JDK-8230920 : jpackage problems when -input dir contains any files with "cfg" extension.

Alexey Semenyuk alexey.semenyuk at oracle.com
Mon Sep 30 12:01:56 UTC 2019


Looks good.

- Alexey

On 9/30/2019 7:31 AM, Andy Herrick wrote:
> Please review the revised jpackage fix for bug [1] at [4].
>
> This is a fix for the JDK-8200758-branch branch of the open sandbox 
> repository (jpackage).
>
> I have removed the caching in AppImageFile and moved it to the caller, 
> in this case WinMsiBundler, where the knowledge resides that the value 
> of the arg to AppImageFile.load() is not changeable .
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8230920
>
> [4] http://cr.openjdk.java.net/~herrick/8230920/webrev.04/
>
> /Andy
>
> On 9/29/2019 4:06 PM, Alexey Semenyuk wrote:
>> One thing that I just notice is that you introduced caching in 
>> AppImageFile file. The caching doesn't take into consideration that 
>> different values can be passed in AppImageFile.get() function. So if 
>> the function would be called multiple times with different values of 
>> `appImageDir` parameter the same AppImageFile instance will be 
>> returned in all the calls. Unfortunately unit tests didn't catch 
>> this. I suggest you either fix the caching logic or get rid of it in 
>> AppImageFile class.
>>
>> - Alexey
>>
>> On 9/29/2019 10:37 AM, Andy Herrick wrote:
>>> Yes  - I meant webrev.03
>>>
>>> Sorry
>>>
>>> /Andy
>>>
>>> On 9/29/19 10:23 AM, Alexey Semenyuk wrote:
>>>> I guess it was a type in webrev adderss. Should be 
>>>> http://cr.openjdk.java.net/~herrick/8230920/webrev.03/
>>>>
>>>> This webrev looks good.
>>>>
>>>> - Alexey
>>>>
>>>> On 9/29/2019 10:08 AM, Andy Herrick wrote:
>>>>> Please review the revised jpackage fix for bug [1] at [3].
>>>>>
>>>>> This is a fix for the JDK-8200758-branch branch of the open 
>>>>> sandbox repository (jpackage).
>>>>>
>>>>> The revised fix stores an xml file (.jpackage.xml) in the 
>>>>> app-image so that jpackage commands using that app-image can 
>>>>> determine some things previously available only when building the 
>>>>> app-image, such as the name of the application and the name of 
>>>>> additional launchers.
>>>>>
>>>>> Initially this is only used on windows, but should be later used 
>>>>> on linux and possibly macOS as well (if additional data needed by 
>>>>> dmg or pkg building are identified).
>>>>>
>>>>> The windows fix also ensures that shortcuts are created for all 
>>>>> launchers (when shortcut hint option(s) are used).
>>>>>
>>>>> [1] https://bugs.openjdk.java.net/browse/JDK-8230920
>>>>>
>>>>> [3] http://cr.openjdk.java.net/~herrick/8230920/webrev.01/
>>>>>
>>>>> /Andy
>>>>>
>>>>> On 9/24/2019 8:54 AM, Andy Herrick wrote:
>>>>>> Please review the jpackage fix for bug [1] at [2].
>>>>>>
>>>>>> This is a fix for the JDK-8200758-branch branch of the open 
>>>>>> sandbox repository (jpackage).
>>>>>>
>>>>>> This fix replaces the practice we were using on windows to 
>>>>>> determine the application name, and name of additional launchers 
>>>>>> (by looking for ".cfg" files in the app dir.
>>>>>>
>>>>>> Instead we now add a file ".jpackage.args" to the root of the 
>>>>>> app-image, and record in that file all the arguments used to 
>>>>>> create the app-image.  We later read that file to determine the 
>>>>>> original app name and any additional launcher names.
>>>>>>
>>>>>> This change also fixes the shortcut creation on windows to create 
>>>>>> shortcuts (if so directed) for all launchers.
>>>>>>
>>>>>> [1] https://bugs.openjdk.java.net/browse/JDK-8230920
>>>>>>
>>>>>> [2] http://cr.openjdk.java.net/~herrick/8230920/webrev.01/
>>>>>>
>>>>>> /Andy
>>>>>>
>>>>
>>



More information about the core-libs-dev mailing list