RFR: JDK-8245831: Unify code parsing version strings on Mac and Windows

Alexey Semenyuk alexey.semenyuk at oracle.com
Fri May 29 18:06:42 UTC 2020



On 5/27/2020 10:05 AM, Andy Herrick wrote:
> looks good
>
> curious why app.identifier was added to cfg file.  I don't see it used 
> anywhere.
Right. That is why I removed it.
Thank you for the review!

- Alexey

>
> /Andy
>
> On 5/26/2020 3:26 PM, Alexey Semenyuk wrote:
>> Please review fix [2] for jpackage bug [1].
>>
>> Get rid of duplicated code parsing version strings. Move the code 
>> parsing version strings to dedicated classes with unit test coverage. 
>> Also remove Mac specific identifier setting in app's config file from 
>> the shared code.
>>
>> - Alexey
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8245831
>>
>> [2] http://cr.openjdk.java.net/~asemenyuk/8245831/webrev.00
>>



More information about the core-libs-dev mailing list