rustup should not delete the entire ~/.rustup/tmp dir
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 7k
- Forks
- 1.1k
- Avg merge
- 22h 40m
- Merged PRs (30d)
- 46
Description
Rustup should only remove the tmp/<toolchain> dir that it downloaded
When rustup-toolchain-install-master (RTIM) and rustup update run concurrently, rustup often finishs
installation first (probably because of nightly toolchain is cache by cloudfont, I don't know), so
the tmp is removed by rustup. This cause RTIM fails to install the master toolchain.
Describe the solution you'd like
I don't have a clear suggestion. But I think rustup could keep the tmp dir, and only remove
the toolchain that it is going to install.
Notes
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing concurrent rustup update and rustup-toolchain-install-master runs, then trace the cleanup of ~/.rustup/tmp. Done means rustup removes only the tmp/ directory it downloaded and concurrent installation no longer fails.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100