RFR: JDK-8235453: tools/jpackage/junit/junit.java failed due to "module not found: jdk.incubator.jpackage"

[Alexander Matveev]

Looks good, but is there better way to exclude tests on Solaris? I do 
not like idea adding @requires for all tests.

Thanks,
Alexander

On 12/6/2019 10:35 AM, Alexey Semenyuk wrote:
> Looks good.
>
> - Alexey
>
> On 12/6/2019 1:33 PM, Andy Herrick wrote:
>> Please review this jpackager test fix for bug [1] at [2].
>>
>> the fix adds "@requires (os.family == "linux") | (os.family == "mac") 
>> | (os.family == "windows")" to all jpackage tests that were not 
>> already filtered with "@requires (os.family == "XXX")"
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8235453
>>
>> [2] http://cr.openjdk.java.net/~herrick/8235453/
>>
>> /Andy
>>
>