[9] RFR: 8159781: jlink --include-locales fails with java.util.regex.PatternSyntaxException

[Naoto Sato]

Decided to fix this separately from the other include locales issues. 
Here is the bug and the proposed fix:

https://bugs.openjdk.java.net/browse/JDK-8159781
http://cr.openjdk.java.net/~naoto/8159781/webrev.00/

The change is to explicitly specify "regex" path matching along with the 
pattern modifications. This is needed to cope with the fix to 8158402.

Please review.

Naoto

On 17/06/2016 09:09, Naoto Sato wrote:
> I will take care of that along with other include locales issues.
>
> Naoto
>
> On 17/06/2016 01:14, Masayoshi Okutsu wrote:
>> I've been waiting for JDK-8159206 to be fixed. But this test still fails
>> after the JDK-8159206 fix. I've filed JDK-8159781 for the failure and
>> pushed this test fix with the following ProblemList.txt change:
>>
>> diff --git a/test/ProblemList.txt b/test/ProblemList.txt
>> --- a/test/ProblemList.txt
>> +++ b/test/ProblemList.txt
>> @@ -387,7 +387,7 @@
>>
>>  # core_tools
>>
>> -tools/jlink/plugins/IncludeLocalesPluginTest.java               8158272
>> generic-all
>> +tools/jlink/plugins/IncludeLocalesPluginTest.java               8159781
>> generic-all
>>
>>  tools/jlink/JLinkOptimTest.java                                 8159264
>> generic-all
>>
>>
>> This test passes with b122  which has the CLDR upgrade.
>>
>> Masayoshi
>>
>> On 6/14/2016 5:18 AM, Alan Bateman wrote:
>>> On 13/06/2016 16:02, Mandy Chung wrote:
>>>
>>>>
>>>> I see.  I’m fine with what you have.  We should enhance the jlink
>>>> testlibrary to run java from a run-time image created for a test to
>>>> run.
>>>>
>>> I'm okay with it too.
>>>
>>> -Alan
>>