Review Request 8205112: Remove non-existent modules from test/jdk/jdk/modules/etc/UpgradeableModules.java

[mandy chung]

This patch cleans up the list of upgradeable modules kept in
test/jdk/jdk/modules/etc/UpgradeableModules.java test verification and
take out the modules that have been removed.

thanks
Mandy

--- a/test/jdk/jdk/modules/etc/UpgradeableModules.java
+++ b/test/jdk/jdk/modules/etc/UpgradeableModules.java
@@ -44,14 +44,8 @@
public class UpgradeableModules {
private static final List<String> UPGRADEABLE_MODULES =
List.of("java.compiler",
- "java.jnlp",
"jdk.internal.vm.compiler",
- "jdk.internal.vm.compiler.management",
- "jdk.deploy",
- "jdk.javaws",
- "jdk.plugin",
- "jdk.plugin.dom"
- );
+ "jdk.internal.vm.compiler.management");

public static void main(String... args) {
Set<String> hashedModules = hashedModules();

[Lance Andersen]

+1

This patch cleans up the list of upgradeable modules kept in test/jdk/jdk/modules/etc/UpgradeableModules.java test verification and take out the modules that have been removed.
thanks
Mandy
--- a/test/jdk/jdk/modules/etc/UpgradeableModules.java
+++ b/test/jdk/jdk/modules/etc/UpgradeableModules.java
@@ -44,14 +44,8 @@
public class UpgradeableModules {
private static final List<String> UPGRADEABLE_MODULES =
List.of("java.compiler",
- "java.jnlp",
"jdk.internal.vm.compiler",
- "jdk.internal.vm.compiler.management",
- "jdk.deploy",
- "jdk.javaws",
- "jdk.plugin",
- "jdk.plugin.dom"
- );
+ "jdk.internal.vm.compiler.management");
public static void main(String... args) {
Set<String> hashedModules = hashedModules();

<Loading Image...>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
***@oracle.com <mailto:***@oracle.com>

[Kevin Rushforth]

Looks good to me.

-- Kevin

Post by mandy chung
This patch cleans up the list of upgradeable modules kept in
test/jdk/jdk/modules/etc/UpgradeableModules.java test verification and
take out the modules that have been removed.
thanks
Mandy
--- a/test/jdk/jdk/modules/etc/UpgradeableModules.java
+++ b/test/jdk/jdk/modules/etc/UpgradeableModules.java
@@ -44,14 +44,8 @@
 public class UpgradeableModules {
     private static final List<String> UPGRADEABLE_MODULES =
         List.of("java.compiler",
-                "java.jnlp",
                 "jdk.internal.vm.compiler",
-                "jdk.internal.vm.compiler.management",
-                "jdk.deploy",
-                "jdk.javaws",
-                "jdk.plugin",
-                "jdk.plugin.dom"
-                );
+                "jdk.internal.vm.compiler.management");
     public static void main(String... args) {
         Set<String> hashedModules = hashed

[Alan Bateman]

Post by mandy chung
This patch cleans up the list of upgradeable modules kept in
test/jdk/jdk/modules/etc/UpgradeableModules.java test verification and
take out the modules that have been removed.

Looks okay to me.

-Alan