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 uninstall doesn't uninstall #5916

Closed
brson opened this issue Aug 21, 2018 · 0 comments · Fixed by #5927
Closed

cargo uninstall doesn't uninstall #5916

brson opened this issue Aug 21, 2018 · 0 comments · Fixed by #5927
Labels

Comments

@brson
Copy link
Contributor

brson commented Aug 21, 2018

When in a source directory, I expect cargo uninstall to remove the compiled package from ~/.cargo/bin. It doesn't seem to do so.

In this example I start with no uutils, cargo install, have uutils, cargo uninstall, still have uutils:

brian@DESKTOP-UCV672I:~/coreutils⟫ ls ~/.cargo/bin
basic-http-server  cargo-benchcmp  cargo-fuzz      cargo-upgrade  mdbook  rustc    rustfmt-format-diff  rustup  wasm2es6js                wasm-pack
cargo              cargo-clippy    cargo-generate  comrak         rg      rustdoc  rust-gdb             tokei   wasm-bindgen
cargo-add          cargo-fmt       cargo-rm        git-fmt        rls     rustfmt  rust-lldb            twiggy  wasm-bindgen-test-runner
brian@DESKTOP-UCV672I:~/coreutils⟫ cargo +nightly install
warning: To build the current package use `cargo build`, to install the current package run `cargo install --path .`
  Installing uutils v0.0.1 (file:///home/brian/coreutils)
    Finished release [optimized] target(s) in 0.95s
  Installing /home/brian/.cargo/bin/uutils
brian@DESKTOP-UCV672I:~/coreutils⟫ ls ~/.cargo/bin
basic-http-server  cargo-benchcmp  cargo-fuzz      cargo-upgrade  mdbook  rustc    rustfmt-format-diff  rustup  uutils        wasm-bindgen-test-runner
cargo              cargo-clippy    cargo-generate  comrak         rg      rustdoc  rust-gdb             tokei   wasm2es6js    wasm-pack
cargo-add          cargo-fmt       cargo-rm        git-fmt        rls     rustfmt  rust-lldb            twiggy  wasm-bindgen
brian@DESKTOP-UCV672I:~/coreutils⟫ cargo +nightly uninstall
brian@DESKTOP-UCV672I:~/coreutils⟫ ls ~/.cargo/bin
basic-http-server  cargo-benchcmp  cargo-fuzz      cargo-upgrade  mdbook  rustc    rustfmt-format-diff  rustup  uutils        wasm-bindgen-test-runner
cargo              cargo-clippy    cargo-generate  comrak         rg      rustdoc  rust-gdb             tokei   wasm2es6js    wasm-pack
cargo-add          cargo-fmt       cargo-rm        git-fmt        rls     rustfmt  rust-lldb            twiggy  wasm-bindgen

Tested on stable and nightly.

bors added a commit that referenced this issue Aug 23, 2018
Make "cargo uninstall" uninstall the cwd bins

Fixes #5916

Tested with a local build of cargo, using coreutils:

    17:33:57 $ dcargo uninstall
    Removing /Users/dnw/.cargo/bin/uutils
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants