system76 / ec

System76 Open Source Embedded Controller
GNU General Public License v3.0
318 stars 72 forks source link

scripts: Remove explicit rustup self update #445

Closed crawfxrd closed 5 months ago

crawfxrd commented 6 months ago

Commit 17f8e37ed578 ("Convert toolchain file to TOML syntax") switched from the bare toolchain file to the TOML-based one for better management of the toolchain and components used.

Commit 1cb61e69186d ("deps.sh: Update rustup or inform user of env vars") added an explicit rustup self update because there were still cases, a year later, of people not having a rustup new enough to support the TOML-based toolchain file.

Now 2 years after that, it should be safe to drop the explicit self update. The TOML format has widespread adoption and rustup now self updates by default. This should allow distro-provided rustup, which disables the self update feature, to work if it is already installed in place of the one downloaded from https://rustup.rs.