Skip to content

Problem: top folder rust-toolchain makes cargo error #153

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

Closed
leejw51crypto opened this issue Feb 11, 2022 · 3 comments · Fixed by #161
Closed

Problem: top folder rust-toolchain makes cargo error #153

leejw51crypto opened this issue Feb 11, 2022 · 3 comments · Fixed by #161
Assignees
Labels
good first issue Good for newcomers

Comments

@leejw51crypto
Copy link
Collaborator

error: the 'cargo' binary, normally provided by the 'cargo' component, is not applicable to the '1.58.1-aarch64-apple-darwin' toolchain

on apple m1 macs

@stevenatcrypto
Copy link
Collaborator

Look into it.
It seems that we should not restrict to specify a version of Rust in rust-toolchain. Since it cannot support all platforms.
I will remove the file rust-toolchain if cannot find a better method. Thanks.

@stevenatcrypto
Copy link
Collaborator

stevenatcrypto commented Feb 11, 2022

Hi @leejw51crypto , I try to find similar issue in rustup.

I am sorry that I don't have an apple m1 mac. Could you help run the below commands to see if it could work? Thanks.

rustup component remove cargo
rustup component add cargo

I will remove the rust-toolchain file if it cannot work.

@stevenatcrypto
Copy link
Collaborator

I suppose to add rust-version = "1.57" to all Cargo.toml (as Cargo.toml in wasm) to fix this issue. Thanks.

tomtau pushed a commit that referenced this issue Feb 15, 2022
Solution: Restrict minimum Rust version for all sub-crates by adding rust-version = "1.57" to package section in Cargo.toml. Test with Rust 1.57.0 by asdf.
I think the issue #153 may also associate with this rustup-issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants