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

Fix "proc macro not expanded" errors #460

Closed
wants to merge 3 commits into from
Closed

Conversation

nathansobo
Copy link
Contributor

@nathansobo nathansobo commented Feb 19, 2022

Fixes #428

Based on this conversation, it seems like we may need to explicitly enable procedural macro support when initializing rust-analyzer. Although it's strange because there's also this PR which claims to enable support by default.

It also seems like we need to configure rust-analyzer with cargo.loadOutDirsFromCheck: true. When I make both of these configuration changes, I think I'm no longer seeing these warnings. But I'm still not totally confident because I thought it was working before I made the out dirs change and then I ended up seeing the errors again.

This seems to eliminate "proc macro X not expanded" errors. Seems like we needed to explicitly enable it.
@nathansobo
Copy link
Contributor Author

Ugh, now I just tried again and I'm seeing the errors still. So still confused. Maybe this doesn't work after all.

@nathansobo nathansobo changed the title Fix "proc macro not expanded" errors by enabling proc macro support when initializing rust-analyzer Fix "proc macro not expanded" errors Feb 19, 2022
@nathansobo
Copy link
Contributor Author

This isn't the right path, as @as-cii has discovered that this is related to trying to load dynamic libraries in rust-analyzer after it's been code-signed.

@nathansobo nathansobo closed this Feb 20, 2022
@nathansobo nathansobo deleted the fix-proc-macro-errors branch February 20, 2022 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spurious diagnostics for procedural macros not being expanded
1 participant