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

Cargo install uses rustup override for current folder #7312

Closed
johalun opened this issue Aug 30, 2019 · 2 comments
Closed

Cargo install uses rustup override for current folder #7312

johalun opened this issue Aug 30, 2019 · 2 comments
Labels
A-rustup Area: rustup interaction C-bug Category: bug Command-install S-needs-design Status: Needs someone to work further on the design for the feature or fix. NOT YET accepted.

Comments

@johalun
Copy link

johalun commented Aug 30, 2019

Problem

cargo install seem to use the rust version specified with rustup override for the current folder.

Steps

  1. rustup override set stable
  2. cargo install xxx (builds with stable)
  3. rustup override set nightly
    4 cargo install xxx (builds with nightly)

Possible Solution(s)

I think it would make more sense to use the default toolchain.

Notes

Output of cargo version: cargo 1.39.0-nightly (22f7dd0 2019-08-27)

Rust stable 1.37 and nightly 1.39.
FreeBSD 13-CURRENT.

@johalun johalun added the C-bug Category: bug label Aug 30, 2019
@ehuss ehuss added A-rustup Area: rustup interaction Command-install labels Sep 21, 2019
@steffahn
Copy link
Member

steffahn commented Feb 8, 2022

This behavior is insane. I’ve been using Rust for 2 years now and never noticed this until today when a cargo install failed in a directory with an old version in the override. So I’ve been unintentionally occasionally installing tools with nightly without even noticing!

@weihanglo weihanglo added the S-needs-design Status: Needs someone to work further on the design for the feature or fix. NOT YET accepted. label May 24, 2023
@epage
Copy link
Contributor

epage commented Jan 16, 2024

While this is the original, closing in favor of #11036 since it has more of a conversation. Technically, they were created for different ways within rustup of setting the override (local file, user config) but I think we should look at them holistically which means one issue.

@epage epage closed this as not planned Won't fix, can't repro, duplicate, stale Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rustup Area: rustup interaction C-bug Category: bug Command-install S-needs-design Status: Needs someone to work further on the design for the feature or fix. NOT YET accepted.
Projects
None yet
Development

No branches or pull requests

5 participants