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

panic with 'Source should be ready' #366

Open
Noah-Kennedy opened this issue Aug 28, 2023 · 5 comments
Open

panic with 'Source should be ready' #366

Noah-Kennedy opened this issue Aug 28, 2023 · 5 comments

Comments

@Noah-Kennedy
Copy link

An invocation of cargo outdated -w -R is currently failing consistently for me with the following message:

thread 'main' panicked at 'Source should be ready', /home/noah/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cargo-outdated-0.13.1/src/cargo_ops/temp_project.rs:394:18

This happens for any repo I have tried that uses a specific third-party sparse-only registry we currently run internally (a forked version of alexandrie which serves sparse).

I can reproduce this consistently from my laptop, however I do not have an MCRE that I can share on here.

As the author of another (this one open source) sparse-only registry that we are going to be migrating to soon, I can try and use that to get an MCRE later if you'd like.

@Noah-Kennedy
Copy link
Author

@Noah-Kennedy
Copy link
Author

I have yet to try this with other third party or sparse-only registries.

@Noah-Kennedy
Copy link
Author

@kbknapp I don't quite follow what's going on in this code segment. What's the expected state of things at the time that this panic occurs, and what is going wrong here exactly that cargo-outdated doesn't like?

@MikeDevresse
Copy link

I have the same issue and it seems that commenting the code fragment that you mentioned solve the issue. It seems that the block_until_ready doesn't do its job and that the invalidate_cache is not necessary because the repository is updated anyway.

@hseeberger
Copy link

Same issue here.

jrudolph added a commit to jrudolph/cargo-outdated that referenced this issue May 28, 2024
As reported in kbknapp#366 this previously failed with a panic and
message "Source should be ready". Now, private registries still
cannot be queried but after a timeout (it seems), those dependencies
from private registries are just skipped (with a message "Update for xyz
cannot be found").
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

No branches or pull requests

3 participants