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

cargo upgrade gets stuck when using sparse registry protocol #868

Closed
amsam0 opened this issue Aug 8, 2023 · 5 comments · Fixed by #907
Closed

cargo upgrade gets stuck when using sparse registry protocol #868

amsam0 opened this issue Aug 8, 2023 · 5 comments · Fixed by #907

Comments

@amsam0
Copy link

amsam0 commented Aug 8, 2023

Hi, running cargo upgrade gets stuck when using the sparse registry protocol. (It may be cloning the entire registry, but I'd rather not wait to find out)

The issue is caused here: https://github.com/killercup/cargo-edit/blob/master/src/fetch.rs#L322
cargo-edit uses 0.x of crates-index, which only supports the git registry protocol.

To fix this, all that's needed is to upgrade crates-index to the latest version (2.x) and then detect the registry protocol and use the corresponding Index struct in crates-index.

Some of crates-index's features will need to be enabled: https://docs.rs/crates-index/latest/crates_index/index.html#feature-flags
To determine the struct to use, we need to get the value of this config option: https://doc.rust-lang.org/cargo/reference/config.html#registriescrates-ioprotocol
If the protocol is sparse, this struct should be used: https://docs.rs/crates-index/latest/crates_index/struct.SparseIndex.html
If the protocol is git, this struct should be used: https://docs.rs/crates-index/latest/crates_index/struct.GitIndex.html

@coriolinus
Copy link

Anyone have a recommended workaround for now?

@epage
Copy link
Collaborator

epage commented Jun 14, 2024

I believe cargo +nightly -Zunstable-options update --breaking should be in the latest nightly.

@VorpalBlade
Copy link

Based on rust-lang/cargo#14204 it seems that cargo update --breaking is in fact not meant to be a replacement for cargo upgrade and there is no plan to support that either.

@coriolinus
Copy link

That's unfortunate. cargo upgrade was quite useful.

Never expected to find myself on this side of https://xkcd.com/1172/.

@epage
Copy link
Collaborator

epage commented Jul 6, 2024

From another issue they posted on

It is the replacement we are offering. It is not a direct port but what we found works for integrating into cargo.

If people have feedback on the design, the place to do it is on the primary issue for --breaking. Please read the first couple of comments from me on it so your feedback can be in reply to them rather than retreading existing ground.

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

Successfully merging a pull request may close this issue.

4 participants