<https://internals.rust-lang.org/t/announcing-rust...
# developmentw
witty-crayon-22786
11/26/2018, 8:19 PMhttps://internals.rust-lang.org/t/announcing-rust-2018-beta-release/8901 ... i might start this tonight, if @average-vr-56795 hasn't already?
a
average-vr-56795
11/26/2018, 9:36 PMI last tried beta 2 IIRC
average-vr-56795
11/26/2018, 9:38 PMThe big thing that didn't Just Work with autofixes was that extern crates got removed but the attributes attached to them didn't, which broke some things; I got rid of a lot of our crate attributes, and I assume they've fixed that bug
w
witty-crayon-22786
11/26/2018, 9:38 PMwould you be in favor of pinning to this release?
a
average-vr-56795
11/26/2018, 9:39 PMFeel free to have a play :) I have a local branch somewhere but it was basically generated by running cargo fix a bunch of times
w
witty-crayon-22786
11/26/2018, 9:39 PM...or, hm. is the idea that we would wait for
1.31witty-crayon-22786
11/26/2018, 9:40 PMoh. i think i get it. "help test the beta" means, upgrade to a 1.31 candidate, which represents the expected final first 2018
witty-crayon-22786
11/26/2018, 9:40 PMso maybe we don't upgrade quite yet, but we attempt the upgrade and post it somewhere to land post 1.31
a
average-vr-56795
11/26/2018, 9:42 PMMay be worth waiting until clippy and rustfmt come out of preview, but sure, works for me. IIRC it goes stable on Dec 6th, which I was intending to update on, but happy to beta for a bit
average-vr-56795
11/26/2018, 9:43 PMAha your messages all came in after I sent mine! Silly undergroundness
average-vr-56795
11/26/2018, 9:44 PMI suspect as the upgrade is basically mechanical, and Borja and I will be making some non-trivial rust changes in the next couple of weeks, probably not worth keeping the branch around
average-vr-56795
11/26/2018, 9:45 PMWe should definitely upgrade when it hits stable - can play around before or not. I will probably do a test update this week if you don't :)
w
witty-crayon-22786
11/26/2018, 9:45 PMok. i'll give it a go tonight if i have time
a
average-vr-56795
11/26/2018, 9:45 PMCool :)
w
witty-crayon-22786
11/26/2018, 9:45 PMthanks
a
average-vr-56795
11/26/2018, 9:46 PMI'm curious about both how many random scopes we can drop because of NLL, and how to identify them without just reading all our code
w
witty-crayon-22786
11/26/2018, 9:46 PMmm, yea. good question.
witty-crayon-22786
11/26/2018, 9:47 PMthere is always some other
goalpost, but i'm realizing that borrowing across async boundaries is going to be the big big thing.
witty-crayon-22786
11/26/2018, 9:47 PMand that's still post async stabilization
a
average-vr-56795
11/26/2018, 9:48 PMThat's the big migration piece I have no clue about. But also, we don't need to drop the extra scopes, it would just be nice to...
w
witty-crayon-22786
11/26/2018, 9:48 PMright
a
average-vr-56795
11/26/2018, 9:49 PMYeah, async borrows will be fancy. On the plus side, they're making good pace at developing that area :)
w
witty-crayon-22786
11/26/2018, 9:49 PMyea, i don't know how many there will be. but they also shouldn't (?) affect performance
witty-crayon-22786
11/26/2018, 9:49 PMindeed! the stabilization rfcs have been lively, but seemingly converging
a
average-vr-56795
11/26/2018, 9:51 PMYeah, no performance, just a code simplicity thing. I believe NLL only applies to the borrow checker, and doesn't change any drop semantics (otherwise scary things would happen), just just a widening of what code is accepted
average-vr-56795
11/26/2018, 11:24 PMI ended up giving the migration a go; filed one bug with them (linked on the above thread)... Will push a branch for the sake of interest :)
average-vr-56795
11/26/2018, 11:34 PMOh dear, I just hit an ICE :(
w
witty-crayon-22786
11/26/2018, 11:39 PMYay! It was all worthwhile =D