Skip to content

Upgrade clap 3 to clap 4#847

Merged
emilio merged 2 commits intomozilla:masterfrom
boxdot:master
Aug 25, 2023
Merged

Upgrade clap 3 to clap 4#847
emilio merged 2 commits intomozilla:masterfrom
boxdot:master

Conversation

@boxdot
Copy link
Copy Markdown
Contributor

@boxdot boxdot commented Jun 19, 2023

This also removes the dependency on the unmaintained crate atty.

Closes #828

This also removes the dependency on the unmaintained crate atty.

Closes mozilla#828
@jschwe
Copy link
Copy Markdown
Contributor

jschwe commented Jun 22, 2023

What is the current minimum supported Rust version for clap 4? Currently the msrv for cbindgen is 1.57, so that would presumably need to be bumped too.

@boxdot
Copy link
Copy Markdown
Contributor Author

boxdot commented Jun 22, 2023

@jschwe Indeed, thank you. I updated the version in Cargo.toml. Is it ok to increase it, or does cbindgen has a hard MSRV requirement?

@jschwe
Copy link
Copy Markdown
Contributor

jschwe commented Jun 22, 2023

As far as I know (but I'm not a maintainer) there currently is no official MSRV policy. I don't know if cbindgen is used in Firefox, but I could imagine the firefox msrv policy applies also to cbindgen:

We will update the Minimum Supported Rust Version (MSRV) when required.

@romanofski
Copy link
Copy Markdown

Hi! Would it possible to merge this and cut a new release?

@romanofski
Copy link
Copy Markdown

Hope I'm not overstepping but pinging @emilio if a merge and release is possible. Sorry otherwise.

@emilio emilio merged commit b61aa2c into mozilla:master Aug 25, 2023
@jschwe
Copy link
Copy Markdown
Contributor

jschwe commented Aug 25, 2023

The PR breaks the "Test package" test in CI (which I can reproduce locally), therefore I propose reverting it for now in #868.

fredszaq added a commit to fredszaq/cbindgen that referenced this pull request Sep 7, 2023
the msrv was bumped in in the Cargo.toml mozilla#847 but not in clippy.toml,
clippy was complaining at startup and defaulting to 1.57
@fredszaq fredszaq mentioned this pull request Sep 7, 2023
fredszaq added a commit to fredszaq/cbindgen that referenced this pull request Sep 7, 2023
the msrv was bumped in in the Cargo.toml mozilla#847 but not in clippy.toml,
clippy was complaining at startup and defaulting to 1.57
emilio pushed a commit that referenced this pull request Sep 8, 2023
the msrv was bumped in in the Cargo.toml #847 but not in clippy.toml,
clippy was complaining at startup and defaulting to 1.57
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.

Upgrade to clap 4 (getting rid of atty as a transitive dependency)

4 participants