-
Notifications
You must be signed in to change notification settings - Fork 6
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Change rationale for manually looking for CcInfo
We have to skip targets not providing CcInfo not just because of incompatible targets in old Bazel versions, but also due to bug: bazelbuild/bazel#19609 Given incompatible targets for aspects have been fixed in recent Bazel versions and the bug above has just been discovered, it makes no sense to keep mentioning incompatible targets as when above bug is fixed and we can remove this manual check for CcInfo incompatible targets are no longer relevant either way.
- Loading branch information
Showing
2 changed files
with
7 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters