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

rustfmt no longer builds after rust-lang/rust#79336 #79407

Closed
rust-highfive opened this issue Nov 25, 2020 · 10 comments · Fixed by #79737
Closed

rustfmt no longer builds after rust-lang/rust#79336 #79407

rust-highfive opened this issue Nov 25, 2020 · 10 comments · Fixed by #79737
Assignees
Labels
A-rustfmt Area: Rustfmt C-bug Category: This is a bug.

Comments

@rust-highfive
Copy link
Collaborator

Hello, this is your friendly neighborhood mergebot.
After merging PR #79336, I observed that the tool rustfmt no longer builds.
A follow-up PR to the repository https://github.com/rust-lang/rustfmt is needed to fix the fallout.

cc @camelid, do you think you would have time to do the follow-up work?
If so, that would be great!

@rust-highfive rust-highfive added A-rustfmt Area: Rustfmt C-bug Category: This is a bug. labels Nov 25, 2020
@camelid
Copy link
Member

camelid commented Nov 25, 2020

Huh, how did this PR break rustfmt? ripgrep doesn't show any use of optin_builtin_traits, and it compiled fine locally.

@camelid
Copy link
Member

camelid commented Nov 25, 2020

Do I need to wait until there's a nightly with the changes or something?

@camelid
Copy link
Member

camelid commented Nov 25, 2020

I think this is the same problem as in #79406. I think this might be a little too beyond my understanding of how to fix for now, so I'm unassigning myself.

@calebcartwright
Copy link
Member

@camelid - rustfmt leverages the internal rustc mods via the auto publish crates process since rustfmt itself is also published to crates.io. As such the version of those rustc mods used within the rustfmt dep tree tend to lag at least a few weeks behind and in broken toolstate cases like this the fix almost always entails updating those crate versions and dealing with all the subsequent fall out of rustc changes within the rustfmt codebase.

I was already in the process of working on such a bump of those crates, so will hopefully have a fix out before too long (though there were some breaking changes in the parser I need to dig into first)

@calebcartwright
Copy link
Member

calebcartwright commented Nov 26, 2020

We'll need a new rustc-ap version to address this

At a minimum the steps will include:

Note that the rustc-ap bumps in racer and rustfmt can be done in parallel (step 3's), the other tasks all have to be sequenced

@camelid
Copy link
Member

camelid commented Nov 26, 2020

next scheduled run is next Tuesday morning, November 1st

I'm assuming you mean December 1st?

@calebcartwright
Copy link
Member

I'm assuming you mean December 1st?

Indeed thank you. Bad copy/pasta job

@camelid
Copy link
Member

camelid commented Nov 26, 2020

Bad copy/pasta job

You're making me hungry ;) 🍝

@Xanewok
Copy link
Member

Xanewok commented Dec 4, 2020

@calebcartwright Racer is updated and published as v2.1.41. Should we wait until rustfmt-nightly is released or should I already bump versions in RLS?

@calebcartwright
Copy link
Member

@Xanewok - I'm actually going to cut a v1.4.29 release for rustfmt that contains some additional things we were able to get merged in the interim. Unfortunately I still don't have access to publish to crates, but will send a PR to RLS with the bumps after i finish in the rustfmt repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rustfmt Area: Rustfmt C-bug Category: This is a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants