nabijaczleweli / cargo-update

A cargo subcommand for checking and applying updates to installed executables
MIT License
1.22k stars 42 forks source link

Handle repositories with more than one binary #75

Closed biluohc closed 6 years ago

biluohc commented 6 years ago
Updating clippy from https://github.com/rust-lang-nursery/rust-clippy
    Updating git repository `https://github.com/rust-lang-nursery/rust-clippy`
error: multiple packages with binaries found: clippy, clippy_workspace_tests    
nabijaczleweli commented 6 years ago

I can hardly see how this is even remotely related to cargo-update?

mati865 commented 6 years ago

@nabijaczleweli if you want to update clippy installed from git you will see this error.

nabijaczleweli commented 6 years ago

That's all cool and good, the same thing happens if you try to install it via raw cargo, so I'm still failing to see how it's cargo-update's fault/problem?

Here's the output from my machine (where I don't have clippy):

$ cargo install --git https://github.com/rust-lang-nursery/rust-clippy
    Updating git repository `https://github.com/rust-lang-nursery/rust-clippy`
error: multiple packages with binaries found: clippy, clippy_workspace_tests

https://github.com/rust-lang-nursery/rust-clippy seems the properer place for this, dunnit

biluohc commented 6 years ago

You can install clippy from git by running the following command:

 cargo install --git https://github.com/rust-lang-nursery/rust-clippy clippy  

Are you willing to deal with this situation?

nabijaczleweli commented 6 years ago

Now we're getting somewhere, that's a fixable issue

nabijaczleweli commented 6 years ago

Acquiesce a check on master, if y'all would. Will release if successful 👍.

nabijaczleweli commented 6 years ago

As a ping on this: if you have anything against the release, speak now, or forever hold your silence (i.e. will release ca. Monday if nobody protests).

mati865 commented 6 years ago

Works fine, thanks.

nabijaczleweli commented 6 years ago

Released in v1.5.0.