This is an archive of the discontinued LLVM Phabricator instance.

[lit] implement 'env' as shell builtin
ClosedPublic

Authored by Tzafrir on Jul 20 2015, 4:09 AM.

Details

Summary

Add implementation of env command as an internal shell builtin.

Diff Detail

Event Timeline

Tzafrir retitled this revision from to [lit] implement 'env' as shell builtin.
Tzafrir updated this object.
Tzafrir added reviewers: yaron.keren, rnk.
Tzafrir added a subscriber: llvm-commits.
rnk edited edge metadata.Jul 20 2015, 9:54 AM

This has a few bugs. 'env' should only affect a single command, and it should only affect one command in the pipeline. The env should be reset for the next command in the pipeline. I'm drafting a patch to handle these edge cases, and I'll let you know when it lands.

This revision was automatically updated to reflect the committed changes.