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
