Skip to content

refactor: configure cargo#1573

Merged
kkysen merged 1 commit into
masterfrom
kkysen/refactor-sparse-registry
Jan 27, 2026
Merged

refactor: configure cargo#1573
kkysen merged 1 commit into
masterfrom
kkysen/refactor-sparse-registry

Conversation

@kkysen
Copy link
Copy Markdown
Contributor

@kkysen kkysen commented Jan 27, 2026

Without this, cargo never tries to load any of the settings from .cargo/config.toml, etc. For example, it doesn't load the [unstable] sparse-registry = true in config.toml set by crisp, which is what is causing the hang in cargo refactor in crisp.

@kkysen kkysen requested a review from ahomescu January 27, 2026 18:19
Copy link
Copy Markdown
Contributor Author

@kkysen kkysen left a comment

Choose a reason for hiding this comment

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

We might also be fetching the whole cargo registry in CI as well (I'm not sure if it's cached), so setting [unstable] sparse-registry = true in .cargo/config.toml might be useful for us as well, but I'll do that as a follow-up.

Without this, `cargo` never tries to load any of the settings from `.cargo/config.toml`, etc.
For example, it doesn't load the `[unstable] sparse-registry = true` in `config.toml` set by crisp,
which is what is causing the hang in `cargo refactor` in crisp.
@kkysen kkysen force-pushed the kkysen/refactor-sparse-registry branch from 2c0c785 to 338d60d Compare January 27, 2026 18:20
@kkysen kkysen merged commit 4b400b4 into master Jan 27, 2026
11 checks passed
@kkysen kkysen deleted the kkysen/refactor-sparse-registry branch January 27, 2026 20:11
kkysen added a commit that referenced this pull request Feb 11, 2026
So #1573 didn't catch all of the `cargo` invocations that
`c2rust-refactor` is doing. This does (I checked all of the places where
`cargo::Config` is created), so it should finally fix
GaloisInc/Tractor-Crisp#55 for good.
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.

2 participants