Skip to content

Fix cargo failures on rust version 1.54#133

Merged
ramnivas merged 3 commits intomainfrom
cargo-update
Jul 30, 2021
Merged

Fix cargo failures on rust version 1.54#133
ramnivas merged 3 commits intomainfrom
cargo-update

Conversation

@ramnivas
Copy link
Contributor

Cargo in the latest version upgraded the server dependency (rust-lang/cargo#9508). As a result, it no longer accepts "" as a version. So we now use a specific version of all such usage.

Cargo in the latest version upgraded the server dependency
(rust-lang/cargo#9508). As a result, it no
longer accepts "" as a version. So we now use a specific version of
all such usage.
@ramnivas ramnivas requested a review from fx-chun July 29, 2021 17:15
Copy link
Contributor

@aykhm aykhm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can also use "*" as the semver if you want to keep some of the versions unspecified like before

@ramnivas
Copy link
Contributor Author

You can also use "*" as the semver if you want to keep some of the versions unspecified like before

I thought about that, but since we use major.minor level of specification for other crates, went with that for consistency.

Copy link
Contributor

@fx-chun fx-chun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM and builds for me.

@ramnivas ramnivas merged commit 73a2927 into main Jul 30, 2021
@ramnivas ramnivas deleted the cargo-update branch August 2, 2021 00:59
shadaj pushed a commit that referenced this pull request Apr 20, 2023
* Fix cargo failures on rust version 1.54

Cargo in the latest version upgraded the server dependency (rust-lang/cargo#9508). As a result, it no longer accepts "" as a version. So we now use a specific version of all such usage.

* Fix new clippy warnings new to Rust 1.54
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.

3 participants