Re: JDK 9 RFR of JDK-8165818: Remove tools/pack200/Pack200Props.java from ProblemList

2016-09-12 Thread Kumar Srinivasan


Looks good. Thanks for looking into this.

Kumar


On 9/12/16 11:38 AM, Amy Lu wrote:

tools/pack200/Pack200Props.java

This test was put into ProblemList.txt in 9/117 due to test timed out 
issue: JDK-8155857.


Test failure (timed out) is reproducible with 9/117 and 9/118, but 
issue gone in 9/119 and *not* reproducible anymore since 9/119. Issue 
must has been resolved with other fixes.


Tested with the very latest build on all platforms, test pass. 
Standalone run test in loop for 500 times, test all pass.


As issue does not exist anymore, test could be removed from 
ProblemList.txt


bug: https://bugs.openjdk.java.net/browse/JDK-8165818

Thanks,
Amy


Sorry, the patch: http://cr.openjdk.java.net/~amlu/8165818/webrev.00/

--- old/test/ProblemList.txt2016-09-12 11:55:51.0 +0800
+++ new/test/ProblemList.txt2016-09-12 11:55:51.0 +0800
@@ -312,8 +312,6 @@

 tools/launcher/FXLauncherTest.java 8068049 linux-all,macosx-all

-tools/pack200/Pack200Props.java 8155857 generic-all
-
 tools/launcher/VersionCheck.java 8165772 generic-all

  







Re: JDK 9 RFR of JDK-8165818: Remove tools/pack200/Pack200Props.java from ProblemList

2016-09-11 Thread Amy Lu

On 9/12/16 11:38 AM, Amy Lu wrote:

tools/pack200/Pack200Props.java

This test was put into ProblemList.txt in 9/117 due to test timed out 
issue: JDK-8155857.


Test failure (timed out) is reproducible with 9/117 and 9/118, but 
issue gone in 9/119 and *not* reproducible anymore since 9/119. Issue 
must has been resolved with other fixes.


Tested with the very latest build on all platforms, test pass. 
Standalone run test in loop for 500 times, test all pass.


As issue does not exist anymore, test could be removed from 
ProblemList.txt


bug: https://bugs.openjdk.java.net/browse/JDK-8165818

Thanks,
Amy


Sorry, the patch: http://cr.openjdk.java.net/~amlu/8165818/webrev.00/

--- old/test/ProblemList.txt2016-09-12 11:55:51.0 +0800
+++ new/test/ProblemList.txt2016-09-12 11:55:51.0 +0800
@@ -312,8 +312,6 @@

 tools/launcher/FXLauncherTest.java 8068049 linux-all,macosx-all

-tools/pack200/Pack200Props.java 8155857 generic-all
-
 tools/launcher/VersionCheck.java 8165772 generic-all

 



JDK 9 RFR of JDK-8165818: Remove tools/pack200/Pack200Props.java from ProblemList

2016-09-11 Thread Amy Lu

tools/pack200/Pack200Props.java

This test was put into ProblemList.txt in 9/117 due to test timed out 
issue: JDK-8155857.


Test failure (timed out) is reproducible with 9/117 and 9/118, but issue 
gone in 9/119 and *not* reproducible anymore since 9/119. Issue must has 
been resolved with other fixes.


Tested with the very latest build on all platforms, test pass. 
Standalone run test in loop for 500 times, test all pass.


As issue does not exist anymore, test could be removed from ProblemList.txt

bug: https://bugs.openjdk.java.net/browse/JDK-8165818

Thanks,
Amy


--- old/test/ProblemList.txt2016-09-12 11:28:05.0 +0800
+++ new/test/ProblemList.txt2016-09-12 11:28:05.0 +0800
@@ -1,4 +1,4 @@
-###
+e##
 #
 # Copyright (c) 2009, 2016, Oracle and/or its affiliates. All rights reserved.
 # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
@@ -312,8 +312,6 @@
 
 tools/launcher/FXLauncherTest.java  8068049 linux-all,macosx-all
 
-tools/pack200/Pack200Props.java 8155857 generic-all

-
 tools/launcher/VersionCheck.java8165772 
generic-all