update README #494
update README #494
Conversation
|
I think this looks great. I didn't follow the removal of the -Zstart-fn stuff, or understand what it was for, so I'll just pretend it never existed. |
|
|
||
| with `+nightly` replaced with the appropriate nightly version of Rust. | ||
| This uses the very latest Rust version. If you experience any problem, refer to | ||
| the `rust-version` file which contains a particular Rust nightly version that |
There was a problem hiding this comment.
We could rename the file to rust-toolchain and rustup would pick it up automatically. Is there any reason we're not doing that?
There was a problem hiding this comment.
lol we just renamed it back from that in the last PR.^^
The reason is that people get really confused when they do cargo install, and then use cargo miri and nothing works because now they have to manually set the nightly to use to exactly match the one miri uses.
Basically, rust-toolchain helps miri developers, but it doesn't help any user that wants to use cargo miri. (It helps for really simple examples that work with cargo run, but you can just as well use the playground for these.) That seems like the wrong tradeoff.
3 participants
@apoelstra what do you think?