Implements setEnv on ExecFuture so that the environment variable can be set.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- rPHUba9c9426ab45: Implement `setEnv`.
Completely untested.
Diff Detail
Diff Detail
- Branch
- setenv
- Lint
Lint Passed - Unit
Tests Passed
Event Timeline
Comment Actions
Oh, actually, I think we should mimic the implementation in PhutilExecPassthru:
public function setEnv(array $env, $wipe_process_env = false) {
if ($wipe_process_env) {
$this->env = $env;
} else {
$this->env = $env + $_ENV;
}
return $this->env;
}This has the side effect of wiping out the current process's ENV, which probably breaks some things.