Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #752 from lmwang/master
[FIXED JENKINS-8614] ProcessTreeTest fails on Windows
  • Loading branch information
jglick committed Apr 8, 2013
2 parents b99a794 + cdc18ec commit 91000fa
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion core/src/main/java/hudson/util/ProcessTree.java
Expand Up @@ -67,6 +67,7 @@

import static com.sun.jna.Pointer.NULL;
import static hudson.util.jna.GNUCLibrary.LIBC;
import static java.util.logging.Level.FINE;
import static java.util.logging.Level.FINER;
import static java.util.logging.Level.FINEST;

Expand Down Expand Up @@ -424,7 +425,17 @@ public synchronized List<String> getArguments() {

@Override
public synchronized EnvVars getEnvironmentVariables() {
if(env==null) env = new EnvVars(p.getEnvironmentVariables());
if(env !=null)
return env;
env = new EnvVars();

try
{
env.putAll(p.getEnvironmentVariables());
} catch (WinpException e)
{
LOGGER.log(FINE, "Failed to get environment variable ", e);
}
return env;
}
});
Expand Down

0 comments on commit 91000fa

Please sign in to comment.