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

[8.1.0] Add no_toolchain_error to the platform rule. #25266

Conversation

katre
Copy link
Member

@katre katre commented Feb 12, 2025

This allows platforms to provide custom error messages to users when no required toolchain can be found for a specific type (similar to the toolchain_type.no_match_error from b60a0f4).

This should point to documentation or directions for users to properly use the platform.

When this is set it replaces the previous message "For more information on platforms or toolchains see https://bazel.build/concepts/platforms-intro."

RELNOTES: Add missing_toolchain_error to the platform rule, to customize error messages when a required toolchain type cannot be found for that platform.

Closes #25256.

This allows platforms to provide custom error messages to users when no required toolchain can be found for a specific type (similar to the `toolchain_type.no_match_error` from
b60a0f4).

This should point to documentation or directions for users to properly use the platform.

When this is set it replaces the previous message "For more information on platforms or toolchains see
https://bazel.build/concepts/platforms-intro."

RELNOTES: Add `missing_toolchain_error` to the `platform` rule, to customize error messages when a required toolchain type cannot be found for that platform.

Closes bazelbuild#25234.

PiperOrigin-RevId: 725775441
Change-Id: Ica00d215ad4ec8fb0edb714369e1d240d52c08f5
@katre katre requested a review from a team as a code owner February 12, 2025 14:09
@github-actions github-actions bot added team-Configurability platforms, toolchains, cquery, select(), config transitions awaiting-review PR is awaiting review from an assigned reviewer labels Feb 12, 2025
@iancha1992 iancha1992 changed the title Add no_toolchain_error to the platform rule. [8.1.0] Add no_toolchain_error to the platform rule. Feb 12, 2025
@iancha1992 iancha1992 self-requested a review February 12, 2025 17:34
@iancha1992
Copy link
Member

@katre @gregestren Looks like this is not a regression in 8.1.0, so according our release policy, we won't create a new rc just for a non-regression bug fix. If we are creating another rc for other reason, I'm open to merge this one, otherwise let's postpone this to the next minor/patch release. Thank you!

@iancha1992 iancha1992 closed this Feb 14, 2025
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Feb 14, 2025
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.

2 participants