commit 0e9149380e2c6b808848611a378c0c3dab2e7908
author adonovan <adonovan@google.com> Mon Mar 23 10:07:17 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Mar 23 10:08:47 2020 -0700
tree 9f6b24ab2d98ac16ebaf2ae48bd579513879a45c
parent cc22385f229077ba5ecd298ecf5a776afbeb919e

bazel packages: restore negation operator carelessly removed by commit ebb86fcdc202c440a362a54f6b0de8a049dfa84b

The effect was to enable a long-disabled license check in Bazel and
disable the check in Blaze.

Fixes github.com/bazelbuild/bazel/issues/10995

PiperOrigin-RevId: 302458676

src/main/java/com/google/devtools/build/lib/packages/RuleFactory.java

diff --git a/src/main/java/com/google/devtools/build/lib/packages/RuleFactory.java b/src/main/java/com/google/devtools/build/lib/packages/RuleFactory.java
index e080fdf..fd50266 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/RuleFactory.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/RuleFactory.java
@@ -125,7 +125,7 @@
       // This flag is overridable by RuleClass.ThirdPartyLicenseEnforcementPolicy (which is checked
       // in RuleClass). This lets Bazel and Blaze migrate away from license logic on independent
       // timelines. See --incompatible_disable_third_party_license_checking comments for details.
-      boolean checkThirdPartyLicenses = semantics.incompatibleDisableThirdPartyLicenseChecking();
+      boolean checkThirdPartyLicenses = !semantics.incompatibleDisableThirdPartyLicenseChecking();
       return ruleClass.createRule(
           pkgBuilder,
           label,