Fixed env-var case bug in Windows.

GoClipse · Dec 29, 2016 · 5dbf00e · 5dbf00e
1 parent cf91dc2
commit 5dbf00e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/documentation/ChangeLog.md b/documentation/ChangeLog.md
@@ -2,6 +2,7 @@
 [Latest features on top]
 
 ### (NextVersion)
+ * Fixed: Build Target's append environment variables might not replace variables correctly in Windows, due to case issues.
  * Fixed stale validation message for location field in New Project Wizard.
  * Fixed stale validation message for Build Command field in Build Targets property page and Launch Configurations dialog.
  * Fixed editor NPE when opening files from a non-workspace VCS revision. 

diff --git a/plugin_tooling/src-lang/melnorme/lang/tooling/commands/CommandInvocation.java b/plugin_tooling/src-lang/melnorme/lang/tooling/commands/CommandInvocation.java
@@ -17,6 +17,7 @@
 import java.util.Map.Entry;
 
 import melnorme.lang.tooling.utils.ArgumentsParser;
+import melnorme.lang.utils.EnvUtils;
 import melnorme.lang.utils.ProcessUtils;
 import melnorme.utilbox.collections.HashMap2;
 import melnorme.utilbox.collections.Indexable;
@@ -106,7 +107,7 @@ public ProcessBuilder getProcessBuilder() {
 			environment.clear();			
 		}
 		for (Entry<String, String> envVar : getEnvironmentVars()) {
-			environment.put(envVar.getKey(), envVar.getValue());
+			EnvUtils.putVarInEnvMap(environment, envVar.getKey(), envVar.getValue());
 		}
 
 		return pb;