diff options
author | Klaus Aehlig <aehlig@google.com> | 2016-08-23 12:25:27 +0000 |
---|---|---|
committer | John Cater <jcater@google.com> | 2016-08-23 22:57:20 +0000 |
commit | 3d4248c6da9630e315f58a18e9d69c2608cd9de5 (patch) | |
tree | 48c5e85c0e208d85d1c2a668065aebc34edaf5c2 /src/test/shell/integration/BUILD | |
parent | 7e1d576614fdf9f3468ec87c444590ae295b03e3 (diff) |
Add option --action_env and make BuildConfiguration declare environment dependencies
This option will allow to specify which environment variables are to
be provided to the actions. Environment variables for the options will
become opt-in, i.e., only environment variables explicitly specified
will be provided to the actions. However, the full range of rc-files
will be able to nominate options to be added; to avoid dependency on
the invocation environment, absolute values can be provided as well.
As the configuration now no longer completely determines the action
environment extend it to also declare which environment variables are
to be taken from the client environment. This will be implemented in
follow-up patches.
The transition plan is that the newly added option takes precedence
over the environment variables added by the fragments. This is conservative,
as the new option is not yet used anywhere. Then the effect of the
fragments will be provided by rc-files, and finally, the setupShellEnvironment
Method will be removed from the fragments all together.
Also add some simple tests for static (i.e., independent of the client
environment) setting of action environment variables.
This is the first step towards implementing the design on [Specifying
environment variables for actions](http://bazel.io/designs/2016/06/21/environment.html).
--
Change-Id: I0ad36913b7d357787b4d69e341926806b3fc61bf
Reviewed-on: https://bazel-review.googlesource.com/#/c/4241
MOS_MIGRATED_REVID=131044391
Diffstat (limited to 'src/test/shell/integration/BUILD')
-rw-r--r-- | src/test/shell/integration/BUILD | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/test/shell/integration/BUILD b/src/test/shell/integration/BUILD index 35958d4ce7..613ed750fc 100644 --- a/src/test/shell/integration/BUILD +++ b/src/test/shell/integration/BUILD @@ -52,6 +52,13 @@ sh_test( data = [":test-deps"], ) +sh_test( + name = "action_env_test", + size = "medium", + srcs = ["action_env_test.sh"], + data = [":test-deps"], +) + test_suite( name = "all_tests", visibility = ["//visibility:public"], |