Skip to content
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

Move to rust 2018 edition #271

Closed
dlrobertson opened this issue Jan 24, 2019 · 8 comments
Closed

Move to rust 2018 edition #271

dlrobertson opened this issue Jan 24, 2019 · 8 comments

Comments

@dlrobertson
Copy link
Collaborator

Get all the latest and greatest by moving to rust 2018 edition.

@whitequark
Copy link
Contributor

This would be a problem for M-Labs because the ARTIQ Rust is not yet at the version that has the edition.

@crawford
Copy link
Contributor

Crates can use independent editions. smoltcp can switch to 2018 while the rest of your code stays on 2015. It works quite well in practice (I'm already using a mix of 2018 and 2015 with smoltcp).

@whitequark
Copy link
Contributor

smoltcp can switch to 2018 while the rest of your code stays on 2015.

The problem is that M-Labs has a fork of rustc which is not recent.

@jbg
Copy link

jbg commented Aug 29, 2019

The problem is that M-Labs has a fork of rustc which is not recent.

(out of interest, how come? Can the changes be contributed upstream?)

Would a PR be accepted for 2018 edition yet, or are you still using an older rustc internally?

@whitequark
Copy link
Contributor

(out of interest, how come? Can the changes be contributed upstream?)

No. M-Labs is using OpenRISC, which has a fork of LLVM, fork of Clang, and fork of rustc to support them both. There are currently neither resources nor people to upstream the OpenRISC backend in LLVM so it is unlikely that this will change unless M-Labs migrates to RISC-V.

However, that fork can be upgraded, which might be the simplest solution.

@hartytp
Copy link

hartytp commented Aug 29, 2019

However, that fork can be upgraded, which might be the simplest solution.

Yes, it would be a shame if forcing smoltcp to use an old version of rust put off wider community usage/contributions.

@Dirbaio
Copy link
Member

Dirbaio commented Dec 25, 2020

ping @whitequark Has this changed since? Should we upgrade to Rust 2018 now?

@whitequark
Copy link
Contributor

Yep, absolutely do go ahead and upgrade.

Dirbaio added a commit that referenced this issue Dec 26, 2020
Dirbaio added a commit that referenced this issue Dec 26, 2020
Dirbaio added a commit that referenced this issue Dec 26, 2020
Dirbaio added a commit that referenced this issue Dec 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

6 participants