Skip to content

Commit 128c0c9

Browse files
author
Alexey Semenyuk
committedMar 29, 2021
8248418: jpackage fails to extract main class and version from app module linked in external runtime
Reviewed-by: herrick, almatvee
1 parent fd45694 commit 128c0c9

File tree

2 files changed

+0
-23
lines changed

2 files changed

+0
-23
lines changed
 

‎test/jdk/ProblemList.txt

-2
Original file line numberDiff line numberDiff line change
@@ -837,8 +837,6 @@ jdk/jfr/startupargs/TestStartDuration.java 8214685 windows-
837837

838838
# jdk_jpackage
839839

840-
tools/jpackage/share/jdk/jpackage/tests/ModulePathTest3.java#id0 8248418 generic-all
841-
842840
############################################################################
843841

844842
# Client manual tests

‎test/jdk/tools/jpackage/share/jdk/jpackage/tests/ModulePathTest3.java

-21
Original file line numberDiff line numberDiff line change
@@ -57,18 +57,6 @@
5757
* --jpt-run=jdk.jpackage.tests.ModulePathTest3
5858
*/
5959

60-
/*
61-
* @test
62-
* @summary jpackage for app's module linked in external runtime
63-
* @library ../../../../helpers
64-
* @build jdk.jpackage.test.*
65-
* @modules jdk.jpackage/jdk.jpackage.internal
66-
* @compile ModulePathTest3.java
67-
* @run main/othervm/timeout=360 -Xmx512m jdk.jpackage.test.Main
68-
* --jpt-run=jdk.jpackage.tests.ModulePathTest3
69-
* --jpt-exclude=test8248418
70-
*/
71-
7260
public final class ModulePathTest3 {
7361

7462
public ModulePathTest3(String jlinkOutputSubdir, String runtimeSubdir) {
@@ -85,15 +73,6 @@ public void test8248254() throws XPathExpressionException, IOException {
8573
testIt("me.mymodule/me.mymodule.Main");
8674
}
8775

88-
/**
89-
* Test case for JDK-8248418.
90-
* App's module with version specified in runtime directory.
91-
*/
92-
@Test
93-
public void test8248418() throws XPathExpressionException, IOException {
94-
testIt("me.mymodule/me.mymodule.Main@3.7");
95-
}
96-
9776
private void testIt(String mainAppDesc) throws XPathExpressionException,
9877
IOException {
9978
final JavaAppDesc appDesc = JavaAppDesc.parse(mainAppDesc);

1 commit comments

Comments
 (1)

openjdk-notifier[bot] commented on Mar 29, 2021

@openjdk-notifier[bot]
Please sign in to comment.