aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
diff options
context:
space:
mode:
authorGravatar jcater <jcater@google.com>2018-06-20 07:56:44 -0700
committerGravatar Copybara-Service <copybara-piper@google.com>2018-06-20 07:59:05 -0700
commitd0c99bd1738c80389acf7b05d72f07eb6015029b (patch)
treef47a88ee49e8bc498131e76b4eddc8553eaecb3a /src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
parent4e578e6f7205a352630720ed482967b6edb4afca (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.java22
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}.