Skip to content
Permalink
Browse files
[JENKINS-18898] Comment.
Unclear what Run.getEnvironment is supposed to return after the build is complete.
In particular, SubversionSCM calls this during polling to substitute variables in e.g. repo URL.
Presumably in such a case node-specific variables such as ToolInstallation.home would be irrelevant.
But in general should this reflect the “current” computer (if any), or the computer on which the build ran?
Originally-Committed-As: 33101a3a01d3ec716a8c81eee97bbccde0c3793e
  • Loading branch information
jglick committed Jul 24, 2013
1 parent c1d194c commit f8b28d60df5ef94c0185381d0879b40eb05eb64b
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/main/java/hudson/maven/MavenModuleSetBuild.java
@@ -172,7 +172,7 @@ public EnvVars getEnvironment(TaskListener log) throws IOException, InterruptedE

Computer computer = Computer.currentComputer();
if (computer != null) { // just in case were not in a build
Node node = computer.getNode();
Node node = computer.getNode(); // TODO should this rather be getBuiltOn()? Cf. JENKINS-18898
if (node != null) {
mvn = mvn.forNode(node, log);
mvn.buildEnvVars(envs);

0 comments on commit f8b28d6

Please sign in to comment.