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.0.0] Put macro attribute inheritance behind an off-by-default --experimental_enable_macro_inherit_attrs flag #24336

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

tetromino
Copy link
Contributor

We still have open questions about how macro attribute inheritance ought to interact with the tracking of whether rule attributes were or were not explicitly provided.

In effect, this re-opens #24066 Part of addressing #24319

RELNOTES: symbolic macro attribute inheritance is now marked experimental; set --experimental_enable_macro_inherit_attrs flag to enable it.

Commit
08beb21

PiperOrigin-RevId: 696582223
Change-Id: I3d7cb434bf8fe2da9cd10019e6075990205e7153

Working towards #24335

…perimental_enable_macro_inherit_attrs flag

We still have open questions about how macro attribute inheritance ought to
interact with the tracking of whether rule attributes were or were not
explicitly provided.

In effect, this re-opens bazelbuild#24066
Part of addressing bazelbuild#24319

RELNOTES: symbolic macro attribute inheritance is now marked experimental; set --experimental_enable_macro_inherit_attrs flag to enable it.

Commit
bazelbuild@08beb21

PiperOrigin-RevId: 696582223
Change-Id: I3d7cb434bf8fe2da9cd10019e6075990205e7153
@tetromino tetromino requested a review from brandjon November 14, 2024 21:19
@tetromino tetromino self-assigned this Nov 14, 2024
@tetromino tetromino requested a review from a team as a code owner November 14, 2024 21:19
@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 Nov 14, 2024
@tetromino tetromino added team-Loading-API BUILD file and macro processing: labels, package(), visibility, glob and removed team-Configurability platforms, toolchains, cquery, select(), config transitions labels Nov 14, 2024
@meteorcloudy meteorcloudy added this pull request to the merge queue Nov 15, 2024
Merged via the queue into bazelbuild:release-8.0.0 with commit 94bdee2 Nov 15, 2024
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 15, 2024
@iancha1992 iancha1992 added this to the 8.0.0 release blockers milestone Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Loading-API BUILD file and macro processing: labels, package(), visibility, glob
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants