-
Notifications
You must be signed in to change notification settings - Fork 925
Move off Syntex #1527
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
Comments
I have a branch that uses libsyntax, not sure if to land or not. Would mean the end of stable rustfmt in the short term. |
There are a few test failures due to #1541 and #1542, otherwise seems to be working. Branch is https://github.com/rust-lang-nursery/rustfmt/tree/libsyntax |
If we don't merge going to have to keep rebasing that branch :-( |
Hi, I creaed a branch (https://github.com/topecongiro/rustfmt/tree/libsyntax) from yours, which fixes #1538, #1541 and #1542 (though just giving up for GlobalAsm and MacroDef). |
Awesome, please PR it! I think we should leave the MacroDef issue open and the FIXME in place - we should at some point fix that properly. |
Thanks! I will create a PR shortly. |
Should I create a PR against master branch? |
No, against libsyntax branch |
Thanks! |
@nrc: ten days ago you published this blog announcing the transition to After some careful experimentation and review, which cost me an hour or so, my conclusion is this:
I personally can imagine some technical tricks get started, but the transition should be documented. I hope you understand this can appear a bit disorganized and disappointing in general. You probably tried to avoid this impression. Can you please elucidate the current status? Do you recognize the issues I point out or not? Thanks for your efforts anyway. |
@sanmai-NL the transition hasn't happened yet. I'll start today and it should be well-documented by the time I'm done. |
@nrc: Thanks for your effort! The time planning was unclear to me from the announcement. |
And use libsyntax directly. This means we stop working on stable Rust. However, we can address this by distributing via Rustup instead of Cargo (#1526).
The text was updated successfully, but these errors were encountered: