OK
-- Jon
On 01/27/2017 01:12 PM, Henry Jen wrote:
How about following patch instead for langtools repo?
diff -r ef142ac9824e test/tools/javac/Paths/wcMineField.sh
--- a/test/tools/javac/Paths/wcMineField.sh Thu Jan 26 16:53:56 2017 -0800
+++ b/test/tools/javac/Paths/wcMineField.sh Fri Jan 27 13:10:12 2017 -0800
@@ -26,7 +26,7 @@
#
# @test
# @summary Test classpath wildcards for javac and java -classpath option.
-# @bug 6268383
+# @bug 6268383 8172309
# @run shell/timeout=600 wcMineField.sh
# To run this test manually, simply do ./wcMineField.sh
@@ -186,6 +186,8 @@
Failure "$javac" ${TESTTOOLVMOPTS} -classpath "GooJar/*${PS}." Main1.java
Success "$javac" ${TESTTOOLVMOPTS} -cp "GooJar/SubDir/*" Main1.java
Success "$javac" ${TESTTOOLVMOPTS} -classpath "GooJar/SubDir/*" Main1.java
+Success "$javac" ${TESTTOOLVMOPTS} --class-path "GooJar/SubDir/*" Main1.java
+Success "$javac" ${TESTTOOLVMOPTS} --class-path="GooJar/SubDir/*" Main1.java
#Same with launcher. Should not load jar in subdirectories unless specified
Failure "$java" ${TESTVMOPTS} -classpath "GooJar/*${PS}." Main1
Success "$java" ${TESTVMOPTS} -classpath "GooJar/SubDir/*${PS}." Main1
Cheers,
Henry
On Jan 26, 2017, at 6:01 PM, Henry Jen <henry....@oracle.com> wrote:
On Jan 26, 2017, at 4:09 PM, Jonathan Gibbons <jonathan.gibb...@oracle.com>
wrote:
Henry,
You've changed old test cases into new test cases, thereby eliminating the old
cases, which is not so good.
You should be adding new test cases, but changing old ones.
I am not sure, I believe all wild-card cases still tested, just that we have
different variety of calling -cp. Of course, it would be nice to test all cases
with all 4 flavors of -cp, but I don’t think that’s necessary as I think the
test coverage is the same.
Anyway, I could be wrong. If you feel strong about this, I can redo it. But
that may takes more time to digest what the test cases are really for.
Cheers,
Henry
-- Jon
On 01/26/2017 02:31 PM, Kumar Srinivasan wrote:
Hi Henry,
Looks ok to me. Thanks for making this change.
Kumar
Hi,
Please review the webrev[1], the fix is to ensure —class-path and —class-path=
is processed correctly to expand wildcard. Changes are made in jdk repo.
However, test case to verify the bug fix is in langtool repo.
Cheers,
Henry
[1] http://cr.openjdk.java.net/~henryjen/jdk9/8172309/0/
[2] https://bugs.openjdk.java.net/browse/JDK-8172309