rust plugin: refactor to use the latest rustup
Metadata
Current evaluation
Merged Rust plugin refactor for latest rustup. Removed deprecated flags, added command qualifiers, established shared rustup/cargo directory, and standardized config. Approved and passed CI.
Suggested action: —
No scores available.
Issue body
- Latest versions of rustup removed --prefix, --disable-sudo and --save.
- Using command qualifiers for channels and revisions.
- A global rustup and cargo area is now in place for multiple parts
to reuse.
- Removed the test target as it was inconsitent with other plugins.
- Made use of .cargo/config for consistency across execution.
LP: #1809259
LP: #1803466
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged Rust plugin refactor for latest rustup. Removed deprecated flags, added command qualifiers, established shared rustup/cargo directory, and standardized config. Approved and passed CI. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a Rust plugin refactor to support the latest rustup. Changes remove deprecated flags, establish a shared rustup and cargo area for multiple parts, and standardize execution via .cargo/config. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a Rust plugin refactor to support the latest rustup. Removed deprecated flags, added a global rustup and cargo area for reuse, and updated configuration for consistency. Successfully integrated. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2431 rust plugin: update to use latest rustup command options | snapcraft | merged | Merged update to the Rust plugin replacing deprecated rustup command options with current equivalents. Approved by a reviewer, passed CI, modified five files, and maintained 94.5% test coverage. | |
| #2438 rust plugin: new link for rustup | snapcraft | merged | Merged into master after approval and passing CI. Updated the rust plugin to use a new rustup link, addressing LP #1662960. Modified 3 files with 11 additions and 6 deletions. | |
| #2538 rust plugin: fix usage of rust-channel | snapcraft | merged | Merged PR #2538 fixes the Rust plugin by replacing the deprecated rustup.sh channel argument with rustup toolchain setup. Approved, passed CI, and merged to master. | |
| #2849 rust plugin: split RUSTUP_HOME and CARGO_HOME | snapcraft | merged | Merged Rust plugin changes decoupling RUSTUP_HOME and CARGO_HOME. Previously both defaulted to ~/.cargo, risking side effects. They now point to ~/.rustup and ~/.cargo respectively. Approved and passed CI. | |
| #560 rust_plugin: multiple fixes | craft-parts | merged | Merged Rust plugin fixes resolving issue #559. Changes remove unnecessary rust_channel override typing checks and improve compiler plugin identification and removal. Approved by reviewers and merged despite an unrelated CI pylint failure. | |
| #530 plugins/rust_plugin: rewrite the plugin to improve UX | craft-parts | merged | Merged a Rust plugin rewrite improving UX with default Rustup, LTO, toolchain version, and feature toggles. Failing tests for Ubuntu 20.04 Chisel support were resolved via a separate PR and rebasing prior to merge. | |
| #2611 rust plugin: use toml to dump the config | snapcraft | merged | Merged update to the Rust plugin using the toml library for config dumping. Approved, passed CI, and modified three files. A trailing comma in the toml requirement was noted but did not block the merge. | |
| #2170 rust plugin: fix cargo builds and run tests | snapcraft | merged | Merged fixes for cargo builds in the Rust plugin along with new tests. The change updated two files with 41 additions and 10 deletions, achieving 100% diff coverage. | |
| #2767 rust plugin: add rustup profile | snapcraft | merged | Merged. Updates the Rust plugin to enforce the minimal rustup profile, resolving build failures where clippy and rustfmt are unavailable. The implementation applies the minimal profile universally and includes updated unit tests. | |
| #570 rust_plugin: multiple fixes (#560) | craft-parts | merged | Merged as #560. The rust_plugin was updated to remove compiler plugins from final Snap packages and eliminate a redundant channel override causing type-checking errors. Approved by one reviewer and passed CI checks. |