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

Improve error when a label is provided in config_setting's values #19464

Closed

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Sep 8, 2023

Suggest to use flag_values instead. Previously, this only resulted in an ambiguous "Unrecognized option" error.

Suggest to use `flag_values` instead. Previously, this only resulted in
an ambiguous "Unrecognized option" error.
@fmeum fmeum requested a review from lberki as a code owner September 8, 2023 20:25
@fmeum fmeum requested review from katre and removed request for lberki September 8, 2023 20:25
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Sep 8, 2023
Copy link
Member

@katre katre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is really handy, thanks!

@katre katre added team-Configurability platforms, toolchains, cquery, select(), config transitions awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Sep 8, 2023
@fmeum
Copy link
Collaborator Author

fmeum commented Sep 8, 2023

@bazel-io flag

@bazel-io bazel-io added the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Sep 8, 2023
@iancha1992
Copy link
Member

@bazel-io fork 6.4.0

@bazel-io bazel-io removed the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Sep 8, 2023
@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Sep 11, 2023
bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Sep 11, 2023
Suggest to use `flag_values` instead. Previously, this only resulted in an ambiguous "Unrecognized option" error.

Closes bazelbuild#19464.

PiperOrigin-RevId: 564367170
Change-Id: Ibe9397be4300783ab6e5b5de9e08e40142de0b4c
@fmeum fmeum deleted the label-in-config-setting-values branch September 11, 2023 14:07
iancha1992 pushed a commit that referenced this pull request Sep 11, 2023
…`values` (#19484)

Suggest to use `flag_values` instead. Previously, this only resulted in
an ambiguous "Unrecognized option" error.

Closes #19464.

Commit
bd5dbc5

PiperOrigin-RevId: 564367170
Change-Id: Ibe9397be4300783ab6e5b5de9e08e40142de0b4c

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
@iancha1992
Copy link
Member

The changes in this PR have been included in Bazel 6.4.0 RC1. Please test out the release candidate and report any issues as soon as possible. If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=last_rc.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Configurability platforms, toolchains, cquery, select(), config transitions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants