Skip to content

Conversation

@rinon
Copy link
Contributor

@rinon rinon commented Sep 26, 2019

We need proper support for parameter attributes, which requires a newer nightly.

I split out the rustc/libsyntax pretty-printer into our own crate so we can once again forge a list of comments to be emitted in the pretty-printed output. It seemed to invasive to request this change upstream, and the pretty-printer should be easy to maintain out-of-tree.

@rinon
Copy link
Contributor Author

rinon commented Sep 26, 2019

Blocked until rust-lang/rust#64793 lands upstream.

@rinon rinon added the blocked Cannot be supported without external support first label Sep 26, 2019
@rinon rinon removed the blocked Cannot be supported without external support first label Oct 1, 2019
@rinon
Copy link
Contributor Author

rinon commented Oct 1, 2019

@TheDan64 The upgrade_ptr_to_ref test is failing, probably because the node ids are all wrong.

@rinon
Copy link
Contributor Author

rinon commented Oct 1, 2019

Make that the ownership_upgrade_ptrs test.

@rinon
Copy link
Contributor Author

rinon commented Oct 3, 2019

Would like to push this to nightly-2019-10-03 to pick up rust-lang/rust#64991. Will test tomorrow.

@rinon rinon self-assigned this Oct 3, 2019
@XVilka
Copy link

XVilka commented Oct 3, 2019

Please, if possible, check if rust-lang/rust-clippy#4590 is included. Since running clippy is a common task after the conversion is done, it's important to not let it fail.

@XVilka
Copy link

XVilka commented Oct 14, 2019

Last buildable nightly toolchain with that ICE fix landed is c27f7568b 2019-10-13. What about using it instead?

@rinon
Copy link
Contributor Author

rinon commented Oct 16, 2019

Last buildable nightly toolchain with that ICE fix landed is c27f7568b 2019-10-13. What about using it instead?

Something changed in the compiler between 2019-10-04 and 2019-10-13 that broke one of our tests. I don't have time to fix it right now, so I'm going to merge this. We can revisit rolling to something newer if/when it is needed.

@rinon rinon merged commit 26be6fb into master Oct 16, 2019
@rinon rinon deleted the nightly-2019-09 branch October 16, 2019 02:40
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.

8 participants