8173711: Disable JAVA_OPTIONS env variable support until JDK-8173712 is resolved
authormchung
Tue, 31 Jan 2017 14:47:57 -0800
changeset 43340 4d25acf48a0b
parent 43339 0d9c9ef833e3
child 43341 da872581fb8e
8173711: Disable JAVA_OPTIONS env variable support until JDK-8173712 is resolved Reviewed-by: ksrini, henryjen
jdk/src/java.base/share/native/libjli/args.c
jdk/test/ProblemList.txt
jdk/test/tools/launcher/I18NArgTest.java
--- a/jdk/src/java.base/share/native/libjli/args.c	Tue Jan 31 21:33:54 2017 +0000
+++ b/jdk/src/java.base/share/native/libjli/args.c	Tue Jan 31 14:47:57 2017 -0800
@@ -429,6 +429,10 @@
 }
 
 jboolean JLI_AddArgsFromEnvVar(JLI_List args, const char *var_name) {
+
+#ifndef ENABLE_JAVA_OPTIONS
+    return JNI_FALSE;
+#else
     char *env = getenv(var_name);
     char *p, *arg;
     char quote;
@@ -515,6 +519,7 @@
     }
 
     return JNI_TRUE;
+#endif
 }
 
 #ifdef DEBUG_ARGFILE
--- a/jdk/test/ProblemList.txt	Tue Jan 31 21:33:54 2017 +0000
+++ b/jdk/test/ProblemList.txt	Tue Jan 31 14:47:57 2017 -0800
@@ -253,6 +253,7 @@
 
 tools/pack200/CommandLineTests.java                             8059906 generic-all
 
+tools/launcher/ArgsEnvVar.java					8173712 generic-all
 tools/launcher/FXLauncherTest.java                              8068049 linux-all,macosx-all
 
 tools/jimage/JImageExtractTest.java                             8169713 generic-all
--- a/jdk/test/tools/launcher/I18NArgTest.java	Tue Jan 31 21:33:54 2017 +0000
+++ b/jdk/test/tools/launcher/I18NArgTest.java	Tue Jan 31 14:47:57 2017 -0800
@@ -96,6 +96,7 @@
         }
 
         // Test via JAVA_OPTIONS
+/*
         Map<String, String> env = new HashMap<>();
         String cmd = "-Dtest.src=" + TEST_SOURCES_DIR.getAbsolutePath() +
                 " -Dtest.classes=" + TEST_CLASSES_DIR.getAbsolutePath() +
@@ -108,6 +109,7 @@
             System.err.println(tr);
             throw new RuntimeException("test fails");
         }
+*/
     }
 
     static void testCharacters(String... args) {