diff options
author | Lukacs Berki <lberki@google.com> | 2016-09-07 14:20:09 +0000 |
---|---|---|
committer | Yun Peng <pcloudy@google.com> | 2016-09-08 08:43:37 +0000 |
commit | 75733e1cceb04f5eaeb3ba821f9c96886af9c0be (patch) | |
tree | 283ec9dad77748fcfd8775a984933a72490636a3 /src/main/java/com/google/devtools/build/lib/rules/python/PyLibrary.java | |
parent | 09cb1a72fecade911b14208801c95a9f4f8ac83a (diff) |
Remove some unused code from the Python rules.
--
MOS_MIGRATED_REVID=132433369
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/python/PyLibrary.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/python/PyLibrary.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/python/PyLibrary.java b/src/main/java/com/google/devtools/build/lib/rules/python/PyLibrary.java index ef1d40f9c3..8d20b9657b 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/python/PyLibrary.java +++ b/src/main/java/com/google/devtools/build/lib/rules/python/PyLibrary.java @@ -23,13 +23,11 @@ import com.google.devtools.build.lib.analysis.RunfilesProvider; import com.google.devtools.build.lib.collect.nestedset.NestedSet; import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder; import com.google.devtools.build.lib.collect.nestedset.Order; -import com.google.devtools.build.lib.packages.RuleClass.ConfiguredTargetFactory.RuleErrorException; import com.google.devtools.build.lib.rules.RuleConfiguredTargetFactory; import com.google.devtools.build.lib.rules.cpp.CcLinkParams; import com.google.devtools.build.lib.rules.cpp.CcLinkParamsProvider; import com.google.devtools.build.lib.rules.cpp.CcLinkParamsStore; import com.google.devtools.build.lib.vfs.PathFragment; - import java.util.ArrayList; import java.util.List; @@ -55,6 +53,9 @@ public abstract class PyLibrary implements RuleConfiguredTargetFactory { List<Artifact> srcs = common.validateSrcs(); List<Artifact> allOutputs = new ArrayList<>(semantics.precompiledPythonFiles(ruleContext, srcs, common)); + if (ruleContext.hasErrors()) { + return null; + } NestedSet<Artifact> filesToBuild = NestedSetBuilder.wrap(Order.STABLE_ORDER, allOutputs); |