diff options
author | John Cater <jcater@google.com> | 2018-03-06 13:18:26 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-03-06 13:20:02 -0800 |
commit | e00a2b191a45e1efd24f2d491b8abd49235c9db1 (patch) | |
tree | 465df1021e49c9831b9123c04889dc826fbc8801 /src/main/java/com/google/devtools/build/lib/rules/platform | |
parent | cf79305f79f627ae20c4da5e832a633fc3994d5a (diff) |
Do not expose platform-related providers to Skylark.
This prevents both creation and access to platform providers from
Skylark.
This is needed so we can load platforms directly from platform-rule
targets without needing a full configured target, and to effiently
distinguish platform providers from non-platform providers.
Change-Id: I6b61f9ee7518d5e9311232908a922596e18fe32f
PiperOrigin-RevId: 188070457
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/platform')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/platform/PlatformCommon.java | 37 |
1 files changed, 0 insertions, 37 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/platform/PlatformCommon.java b/src/main/java/com/google/devtools/build/lib/rules/platform/PlatformCommon.java index 62b3396d77..e68a461390 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/platform/PlatformCommon.java +++ b/src/main/java/com/google/devtools/build/lib/rules/platform/PlatformCommon.java @@ -15,9 +15,6 @@ package com.google.devtools.build.lib.rules.platform; import com.google.devtools.build.lib.analysis.TemplateVariableInfo; -import com.google.devtools.build.lib.analysis.platform.ConstraintSettingInfo; -import com.google.devtools.build.lib.analysis.platform.ConstraintValueInfo; -import com.google.devtools.build.lib.analysis.platform.PlatformInfo; import com.google.devtools.build.lib.analysis.platform.ToolchainInfo; import com.google.devtools.build.lib.packages.Provider; import com.google.devtools.build.lib.skylarkinterface.SkylarkCallable; @@ -43,40 +40,6 @@ public class PlatformCommon { } @SkylarkCallable( - name = PlatformInfo.SKYLARK_NAME, - doc = - "The provider constructor for PlatformInfo. The constructor takes the list of " - + "ConstraintValueInfo providers that defines the platform.", - structField = true - ) - public Provider getPlatformInfoConstructor() { - return PlatformInfo.SKYLARK_CONSTRUCTOR; - } - - @SkylarkCallable( - name = ConstraintSettingInfo.SKYLARK_NAME, - doc = - "The provider constructor for ConstraintSettingInfo. The constructor takes the label that " - + "uniquely identifies the constraint (and which should always be ctx.label).", - structField = true - ) - public Provider getConstraintSettingInfoConstructor() { - return ConstraintSettingInfo.PROVIDER; - } - - @SkylarkCallable( - name = ConstraintValueInfo.SKYLARK_NAME, - doc = - "The provider constructor for ConstraintValueInfo. The constructor takes the label that " - + "uniquely identifies the constraint value (and which should always be ctx.label), " - + "and the ConstraintSettingInfo which the value belongs to.", - structField = true - ) - public Provider getConstraintValueInfoConstructor() { - return ConstraintValueInfo.SKYLARK_CONSTRUCTOR; - } - - @SkylarkCallable( name = ToolchainInfo.SKYLARK_NAME, doc = "The provider constructor for ToolchainInfo. The constructor takes the type of the " |