Skip to content

chore: Fix building the Rust password_manager example#267

Merged
randombit merged 1 commit intomainfrom
jack/fix-cargo-build
Jan 13, 2026
Merged

chore: Fix building the Rust password_manager example#267
randombit merged 1 commit intomainfrom
jack/fix-cargo-build

Conversation

@randombit
Copy link
Copy Markdown
Contributor

The password_manager example failed to build because the backend symlink points to a canister that uses workspace dependency inheritance ({ workspace = true }), but the example's own workspace didn't define [workspace.dependencies].

Fixed by adding the required workspace dependencies to both the example's Cargo.toml and the root Cargo.toml, and updating the canister to use workspace inheritance for ic-vetkeys.

The password_manager example failed to build because the `backend` symlink points
to a canister that uses workspace dependency inheritance ({ workspace = true }),
but the example's own workspace didn't define `[workspace.dependencies]`.

Fixed by adding the required workspace dependencies to both the example's
`Cargo.toml` and the root `Cargo.toml`, and updating the canister to use
workspace inheritance for ic-vetkeys.
@randombit randombit requested a review from a team January 12, 2026 21:22
@randombit
Copy link
Copy Markdown
Contributor Author

This got broken by #252 but was missed due to the CI workflow paths being quite incomplete (missing Cargo.toml and all of the actual Rust backend libraries/canisters, for one thing).

@randombit randombit changed the title Fix building the Rust password_manager example chore: Fix building the Rust password_manager example Jan 12, 2026
Copy link
Copy Markdown
Contributor

@fspreiss fspreiss left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this, @randombit!

@randombit randombit merged commit 4b35be1 into main Jan 13, 2026
19 of 20 checks passed
@randombit randombit deleted the jack/fix-cargo-build branch January 13, 2026 12:34
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