Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.0.0] Add more details to the warning message in --keep_going mode #20450

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

bazel-io
Copy link
Member

@bazel-io bazel-io commented Dec 6, 2023

The current message is too generic and not helpful for the user.

Before:
WARNING: errors encountered while analyzing target '//:bin': it will not be built

---
After:
WARNING: errors encountered while analyzing target '//:bin', it will not be built.
Target //:bin is incompatible and cannot be built, but was explicitly requested.
Dependency chain:
    //:bin (8250b5)   <-- target platform (@@local_config_platform//:host) didn't satisfy constraint @@platforms//:incompatible

Fixes #20443

Commit c5493b9

PiperOrigin-RevId: 588363260
Change-Id: I70e151cd9baa6a0dd7b307b7ddeb9f43ee30b526

The current message is too generic and not helpful for the user.

```
Before:
WARNING: errors encountered while analyzing target '//:bin': it will not be built

---
After:
WARNING: errors encountered while analyzing target '//:bin', it will not be built.
Target //:bin is incompatible and cannot be built, but was explicitly requested.
Dependency chain:
    //:bin (8250b5)   <-- target platform (@@local_config_platform//:host) didn't satisfy constraint @@platforms//:incompatible

```

Fixes bazelbuild#20443

PiperOrigin-RevId: 588363260
Change-Id: I70e151cd9baa6a0dd7b307b7ddeb9f43ee30b526
@bazel-io bazel-io requested a review from a team as a code owner December 6, 2023 11:33
@bazel-io bazel-io added team-Performance Issues for Performance teams awaiting-review PR is awaiting review from an assigned reviewer labels Dec 6, 2023
@joeleba joeleba modified the milestone: 7.0.0 release blockers Dec 6, 2023
@meteorcloudy meteorcloudy changed the title [7.0.0] Bazel 7.0 hides the nature of the errors when run with "--keep_going" [7.0.0] Add more details to the warning message in --keep_going mode Dec 6, 2023
@joeleba joeleba requested a review from meteorcloudy December 6, 2023 12:22
@meteorcloudy meteorcloudy merged commit 2a8c634 into bazelbuild:release-7.0.0 Dec 6, 2023
26 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Performance Issues for Performance teams
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants