diff options
author | 2017-11-21 07:23:28 -0800 | |
---|---|---|
committer | 2017-11-21 07:24:56 -0800 | |
commit | 2d5035685a6180e6abfa7e93d7ee4b7ec0632510 (patch) | |
tree | 8c77eed55891b24c8577f691af9c25f41e2fd32b /src/main/java/com/google/devtools/build/lib/rules/platform/PlatformRule.java | |
parent | 48c9393a4485bcc8388d27658c6c9452d53abcfb (diff) |
Change Platform.remoteExecutionProperties to be a String, not a dict.
This is needed for the ability to synthesize the correct remote
execution protos from a platform rule.
Part of #4128.
Change-Id: I7fa8acf45642a4df4e2beb1ba9c57c2536670486
PiperOrigin-RevId: 176504885
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/platform/PlatformRule.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/platform/PlatformRule.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/platform/PlatformRule.java b/src/main/java/com/google/devtools/build/lib/rules/platform/PlatformRule.java index 74e824f9f6..1abe56accd 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/platform/PlatformRule.java +++ b/src/main/java/com/google/devtools/build/lib/rules/platform/PlatformRule.java @@ -49,9 +49,11 @@ public class PlatformRule implements RuleDefinition { .mandatoryProviders(ImmutableList.of(ConstraintValueInfo.SKYLARK_CONSTRUCTOR.id()))) /* <!-- #BLAZE_RULE(platform).ATTRIBUTE(remote_execution_properties) --> - A key/value dict of values that will be sent to a remote execution platform. + A text proto (the Platform message from + https://github.com/googleapis/googleapis/blob/master/google/devtools/remoteexecution/v1test/remote_execution.proto) + that will be sent to a remote execution platform. <!-- #END_BLAZE_RULE.ATTRIBUTE --> */ - .add(attr(REMOTE_EXECUTION_PROPS_ATTR, Type.STRING_DICT)) + .add(attr(REMOTE_EXECUTION_PROPS_ATTR, Type.STRING)) // Undocumented. Indicates that this platform should auto-configure the platform constraints // based on the current host OS and CPU settings. |