Skip to content

Roll to nightly 2019-09-... - #165

Merged
rinon merged 49 commits into
masterfrom
nightly-2019-09
Oct 16, 2019
Merged

rinon merged 49 commits into
masterfrom
nightly-2019-09

Conversation

@rinon

@rinon rinon commented Sep 26, 2019

Copy link
Copy Markdown
Contributor

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

rinon commented Sep 26, 2019

Copy link
Copy Markdown
Contributor Author

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

rinon commented Oct 1, 2019

Copy link
Copy Markdown
Contributor Author

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

@rinon

rinon commented Oct 1, 2019

Copy link
Copy Markdown
Contributor Author

Make that the ownership_upgrade_ptrs test.

@rinon

rinon commented Oct 3, 2019

Copy link
Copy Markdown
Contributor Author

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

XVilka commented Oct 3, 2019 •

Copy link
Copy Markdown

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

XVilka commented Oct 14, 2019

Copy link
Copy Markdown

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

@rinon

rinon commented Oct 16, 2019 •

Copy link
Copy Markdown
Contributor Author

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.

7 participants