Skip to content

Conversation

r-ml
Copy link
Contributor

@r-ml r-ml commented Aug 21, 2021

Update clap dependency from v3.0.0-beta.2 to v3.0.0-beta.4.

There are breaking changes in the latest clap release that break rust-script's installation, as cargo's dependency resolver assumes that beta.2 and beta.4 are compatible. The result of this is cargo install rust-script fails with multiple

no method named `multiple` found for struct `Arg` in the current scope

To verify this, just clone the repository and run cargo update. You'll see that cargo updates clap and running cargo build fails with the error above.

From v3.0.0-beta.2 to v3.0.0-beta.4
@fornwall fornwall merged commit caff6c8 into fornwall:master Aug 21, 2021
@fornwall
Copy link
Owner

Thanks!

@fornwall
Copy link
Owner

Seems some issues were introduced by the clap upgrade - 32ac67f should fix these, let me know if you spot issues with that!

@r-ml
Copy link
Contributor Author

r-ml commented Aug 21, 2021

Seems some issues were introduced by the clap upgrade - 32ac67f should fix these, let me know if you spot issues with that!

Thanks!, I'm using the scripts with arguments, so rust-script script.rs arg1 and didn't check the no arguments case. Sorry about that 🙏 . I works well now!

@r-ml
Copy link
Contributor Author

r-ml commented Aug 21, 2021

I might rely too much on CI, these days 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants