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

Latest Clap v3 depends on yanked version of textwrap #4418

Closed
2 tasks done
Enet4 opened this issue Oct 23, 2022 · 3 comments
Closed
2 tasks done

Latest Clap v3 depends on yanked version of textwrap #4418

Enet4 opened this issue Oct 23, 2022 · 3 comments
Labels
C-bug Category: Updating dependencies

Comments

@Enet4
Copy link
Contributor

Enet4 commented Oct 23, 2022

Please complete the following tasks

Rust Version

1.64.0

Clap Version

3.2.22

Minimal reproducible code

clap = { version = "3.2.22", features = ["derive"] }

Steps to reproduce the bug with the above code

cargo update

Actual Behaviour

Version 0.15.1 of textwrap has been yanked due to unexpected breaking changes. Due to pull request #4221, this is the only version that matches the requirement ^0.15.1, and as such it becomes impossible to cargo update a project using Clap v3.

Example error message (obtained while working on DICOM-rs):

error: failed to select a version for the requirement `textwrap = "^0.15.1"`
candidate versions found which didn't match: 0.16.0, 0.15.0, 0.14.2, ...
location searched: crates.io index
required by package `clap v3.2.22`
    ... which satisfies dependency `clap = "3.2.22"` of package `dicom-storescp v0.1.0-rc.1 (.../dicom-rs/storescp)`

Expected Behaviour

It should work fine, probably by having clap v3 depend on textwrap v0.16.0.

Additional Context

No response

Debug Output

No response

@Enet4 Enet4 added the C-bug Category: Updating dependencies label Oct 23, 2022
crepererum added a commit to crepererum/clap that referenced this issue Oct 24, 2022
Version 0.15.1 was yanked because it should have been 0.16.

Fixes clap-rs#4418.
crepererum added a commit to crepererum/clap that referenced this issue Oct 24, 2022
Version 0.15.1 was yanked because it should have been 0.16.

Fixes clap-rs#4418.
crepererum added a commit to crepererum/clap that referenced this issue Oct 24, 2022
Version 0.15.1 was yanked because it should have been 0.16.

Fixes clap-rs#4418.
crepererum added a commit to crepererum/clap that referenced this issue Oct 24, 2022
Version 0.15.1 was yanked because it should have been 0.16.

Fixes clap-rs#4418.
@mgeisler
Copy link
Contributor

mgeisler commented Oct 24, 2022

Sorry abou this! I've just release a 0.15.2 release which is identical to 0.15.0 — and thus a safe semver-compatible upgrade path from the yanked 0.15.1. Please see mgeisler/textwrap#484.

@epage
Copy link
Member

epage commented Oct 24, 2022

Thanks!

@epage epage closed this as completed Oct 24, 2022
@Rudxain
Copy link

Rudxain commented Oct 24, 2022

Thanks! All of my workflows (1 workflow per repo) were failing because of this. I appreciate the speed at which this was fixed (other repos would've taken 1 week, lol)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: Updating dependencies
Projects
None yet
Development

No branches or pull requests

4 participants