packaging: pin cross-group deps to git tags behind an umbrella patch#484
Open
mfw78 wants to merge 2 commits into
Open
packaging: pin cross-group deps to git tags behind an umbrella patch#484mfw78 wants to merge 2 commits into
mfw78 wants to merge 2 commits into
Conversation
Every cross-group dep in the videre and shepherd manifests now carries its final form: nexum-* crates pinned to the nexum-runtime repo at v0.1.0, videre-* crates pinned to the videre-nexum-module repo at v0.1.0. A root umbrella [patch] resolves each pinned crate from its in-tree group, so the monorepo builds green before the remotes are populated and the carve reduces to deleting the patch tables. Each group also gains a Cargo.repo.toml: the patch-free workspace root the standalone repo renames to Cargo.toml at the carve, carrying only the hoisted dep-table subset its members inherit.
Resolve the workspace root via the nearest [workspace] Cargo.toml and module.toml sources via the group root, and drop --all-features from the nexum-runtime graph oracle (cargo rejects it for an out-of-workspace dep). The videre-host integration tests now pass both in the monorepo and the carved videre-nexum-module repo.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Repoint every cross-group
nexum-*/videre-*dependency edge from anin-tree path to its eventual git-tag form (
tag = "v0.1.0"), and addan umbrella
[patch]in the rootCargo.tomlthat resolves those tagsback to the in-tree crates until the split repos exist. Adds the
per-group
Cargo.repo.tomlfiles (renamed toCargo.tomlat thecarve) with no cross-group
[patch]of their own.Why
D9/D10 of the split plan: git-tag pins are the cross-repo Rust dep
medium, and the monorepo stays on one workspace with the tags
neutralised by a patch until the physical carve. This is the last
in-tree step before the git-filter-repo cut — every crate's manifest
already reads its final post-carve dependency shape.
Testing
nix develop -c cargo check --workspace --all-featuresnix develop -c cargo fmt --all --checknix develop -c cargo clippy --workspace --all-targets --all-features -- -D warningsscripts/check-venue-agnostic.shAI Assistance
Drafted and gated with Claude Code.