Fixed environment variables injection.

Resolves: https://issues.jenkins-ci.org/browse/JENKINS-19177 Signed-off-by: Oleg Nenashev <nenashev@synopsys.com>
jenkinsci · Sep 4, 2013 · 483d18a · 483d18a
1 parent 73b1e3a
commit 483d18a
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/main/java/com/cloudbees/jenkins/plugins/customtools/CustomToolInstallWrapper.java b/src/main/java/com/cloudbees/jenkins/plugins/customtools/CustomToolInstallWrapper.java
@@ -112,9 +112,8 @@ public void buildEnvVars(Map<String, String> env) {
                 for (CustomTool tool : customTools()) {
                     if (tool.hasVersions()) {
                         ToolVersion version = ToolVersion.getEffectiveToolVersion(tool, buildEnv, node);   
-                        if (version != null && !buildEnv.containsKey(version.getVariableName())) {           
-                            String envStr = version.getVariableName()+"="+version.getDefaultVersion();
-                            buildEnv.addLine(envStr);
+                        if (version != null && !env.containsKey(version.getVariableName())) {
+                            env.put(version.getVariableName(), version.getDefaultVersion());
                         }
                     }
                 }