Skip to content

Commit e8ff528

Browse files
authored
Merge pull request #378 from maven-nar/feature/apply-environment-to-execute
Fix missing setting of the execution environment
2 parents 818e081 + 3f590ef commit e8ff528

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

src/main/java/com/github/maven_nar/cpptasks/CUtil.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -420,7 +420,8 @@ public static int runCommand(final CCTask task, final File workingDir, final Str
420420
exe.setNewenvironment(newEnvironment);
421421
return exe.execute();
422422
*/
423-
return CommandExecution.runCommand(cmdline,workingDir,task);
423+
424+
return CommandExecution.runCommand(cmdline,workingDir,task,env.getVariablesVector());
424425
} catch (final java.io.IOException exc) {
425426
throw new BuildException("Could not launch " + cmdline[0] + ": " + exc, task.getLocation());
426427
}

src/main/java/com/github/maven_nar/cpptasks/CommandExecution.java

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,10 @@
22

33

44
import org.apache.tools.ant.Project;
5+
import org.apache.tools.ant.types.Environment;
56

67
import java.io.*;
8+
import java.util.Vector;
79

810

911
class StreamGobbler extends Thread {
@@ -41,7 +43,7 @@ public class CommandExecution {
4143

4244

4345

44-
public static int runCommand(String[] cmdArgs, File workDir, CCTask task) throws IOException{
46+
public static int runCommand(String[] cmdArgs, File workDir, CCTask task, Vector<Environment.Variable> env) throws IOException{
4547

4648

4749
try {
@@ -54,6 +56,11 @@ public static int runCommand(String[] cmdArgs, File workDir, CCTask task) throws
5456

5557
pb.directory(workDir);
5658

59+
for (Environment.Variable var:env) {
60+
pb.environment().put(var.getKey(), var.getValue());
61+
task.log("Environment variable: " + var.getKey() + "=" + var.getValue(), Project.MSG_VERBOSE);
62+
}
63+
5764
//Start the new process
5865
Process process = pb.start();
5966

0 commit comments

Comments
 (0)