Skip to content

Reject arguments to local_inner_macros for #[macro_export]#155091

Closed
JonathanBrouwer wants to merge 1 commit intorust-lang:mainfrom
JonathanBrouwer:fix-macro-export
Closed

Reject arguments to local_inner_macros for #[macro_export]#155091
JonathanBrouwer wants to merge 1 commit intorust-lang:mainfrom
JonathanBrouwer:fix-macro-export

Conversation

@JonathanBrouwer
Copy link
Copy Markdown
Contributor

Fixes #154977
This is a breaking change, as it adds new cases to a deny-by-default lint.

r? @jdonszelmann

@JonathanBrouwer JonathanBrouwer added S-blocked Status: Blocked on something else such as an RFC or other implementation work. needs-rfc This change is large or controversial enough that it should have an RFC accepted before doing it. needs-crater This change needs a crater run to check for possible breakage in the ecosystem. labels Apr 10, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 10, 2026

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann

@rustbot rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Apr 10, 2026
@JonathanBrouwer
Copy link
Copy Markdown
Contributor Author

@bors try

@rust-bors

This comment has been minimized.

rust-bors bot pushed a commit that referenced this pull request Apr 10, 2026
Reject arguments to `local_inner_macros` for `#[macro_export]`
@jdonszelmann
Copy link
Copy Markdown
Contributor

Alright, this falls under the policy of notify T-lang + crater run, but no FCP right?

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors bot commented Apr 10, 2026

☀️ Try build successful (CI)
Build commit: 52d9d60 (52d9d6051dbe5cf1242df0e063bfcf25e06c91bd, parent: 25a54d4ba87873ef02ef0e5fce49baa57198deb3)

@JonathanBrouwer JonathanBrouwer removed the needs-rfc This change is large or controversial enough that it should have an RFC accepted before doing it. label Apr 10, 2026
@JonathanBrouwer
Copy link
Copy Markdown
Contributor Author

Alright, this falls under the policy of notify T-lang + crater run, but no FCP right?

Right, I think that makes sense, assuming crater finds nothing

@craterbot check

@craterbot
Copy link
Copy Markdown
Collaborator

👌 Experiment pr-155091 created and queued.
🤖 Automatically detected try build 52d9d60
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-crater Status: Waiting on a crater run to be completed. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. S-blocked Status: Blocked on something else such as an RFC or other implementation work. labels Apr 10, 2026
@JonathanBrouwer
Copy link
Copy Markdown
Contributor Author

Closed in favour of #155193
@craterbot abort

@craterbot
Copy link
Copy Markdown
Collaborator

🗑️ Experiment pr-155091 deleted!

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Apr 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-attributes Area: Attributes (`#[…]`, `#![…]`) needs-crater This change needs a crater run to check for possible breakage in the ecosystem. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Invalid value accepted for #[macro_export(local_inner_macros)]

4 participants