Conversation
|
/cc @gregestren Hey, Greg, please take a look at this PR. Do you think we should also propagate |
| effectTags = {OptionEffectTag.ACTION_COMMAND_LINES}, | ||
| help = | ||
| "Specifies the set of environment variables available to actions. " | ||
| "Specifies the set of environment variables available to actions performed using target configuration. " |
There was a problem hiding this comment.
Maybe "with target configuration" sounds a bit simper? Same below.
There was a problem hiding this comment.
To be "available to actions with target configuration." ?
|
I'm going to punt this to @katre. Eventually I think we should move away from needing explicit I also acknowledge we won't have that tomorrow. So the above isn't necessarily an objection to this approach. |
katre
left a comment
There was a problem hiding this comment.
This looks fine with a small fix to the doc comment.
| documentationCategory = OptionDocumentationCategory.OUTPUT_PARAMETERS, | ||
| effectTags = {OptionEffectTag.ACTION_COMMAND_LINES}, | ||
| help = | ||
| "Specifies the set of environment variables available to actions with host configuration. " |
There was a problem hiding this comment.
"with host or execution configurations".
Ideally we'd move away from target/host to non-tool/tool, but we're not there now so no need to burden your PR.
|
Currently, host.actionEnvironment is left unset, correct? Then no need to start propogating the target actionEnvironment to the host configuration. |
|
Thanks for the clarification! Mai, can you fix the documentation? Then you can import this change. |
This PR adds `--host_action_env` option to specify a set of environment variables to be added to the host configuration. The variables specified by this option are only added to host configuration while those specified by `--action_env` are only included in the target configuration. Fixes: bazelbuild#4008 Closes bazelbuild#12122. PiperOrigin-RevId: 333060884
This PR adds
--host_action_envoption to specify a set of environment variables to be added to the host configuration. The variables specified by this option are only added to host configuration while those specified by--action_envare only included in the target configuration.Fixes: #4008