diff options
author | jcater <jcater@google.com> | 2018-06-20 07:56:44 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-06-20 07:59:05 -0700 |
commit | d0c99bd1738c80389acf7b05d72f07eb6015029b (patch) | |
tree | f47a88ee49e8bc498131e76b4eddc8553eaecb3a /src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java | |
parent | 4e578e6f7205a352630720ed482967b6edb4afca (diff) |
Remove genrule's special handling of the "toolchains" attribute.
The general handling should be fine for this.
PiperOrigin-RevId: 201352916
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java | 22 |
1 files changed, 0 insertions, 22 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java b/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java index ecac68f078..d3c2139207 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java @@ -98,11 +98,9 @@ import com.google.devtools.build.lib.util.OrderedSetMultimap; import com.google.devtools.build.lib.util.StringUtil; import com.google.devtools.build.lib.vfs.FileSystemUtils; import com.google.devtools.build.lib.vfs.PathFragment; -import java.util.ArrayList; import java.util.Collection; import java.util.HashMap; import java.util.HashSet; -import java.util.LinkedHashMap; import java.util.LinkedHashSet; import java.util.List; import java.util.Map; @@ -1063,26 +1061,6 @@ public final class RuleContext extends TargetContext return new Expander(this, getConfigurationMakeVariableContext()); } - public ImmutableMap<String, String> getMakeVariables(Iterable<String> attributeNames) { - ArrayList<TemplateVariableInfo> templateVariableInfos = new ArrayList<>(); - - for (String attributeName : attributeNames) { - // TODO(b/37567440): Remove this continue statement. - if (!attributes().has(attributeName)) { - continue; - } - Iterables.addAll(templateVariableInfos, getPrerequisites( - attributeName, Mode.DONT_CHECK, TemplateVariableInfo.PROVIDER)); - } - - LinkedHashMap<String, String> makeVariables = new LinkedHashMap<>(); - for (TemplateVariableInfo templateVariableInfo : templateVariableInfos) { - makeVariables.putAll(templateVariableInfo.getVariables()); - } - - return ImmutableMap.copyOf(makeVariables); - } - /** * Returns a cached context that maps Make variable names (string) to values (string) without any * extra {@link MakeVariableSupplier}. |