Skip to content

Commit

Permalink
Add in-band documentation for zcash#768, since it deviates from typic…
Browse files Browse the repository at this point in the history
…al workspace dependency mgmt.
  • Loading branch information
Nate Wilcox committed Jun 17, 2023
1 parent d2f105e commit 3988578
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 0 deletions.
6 changes: 6 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,12 @@ working with Zcash.

These libraries are currently under development and have not been fully-reviewed.

## Cross-Workspace Dependency Cycle

There is a complication in crate dependencies documented in
[`components/zcash_note_encryption/README.md`](./components/zcash_note_encryption/README.md)
and [issue #768](https://github.com/zcash/librustzcash/issues/768).

## License

All code in this workspace is licensed under either of
Expand Down
9 changes: 9 additions & 0 deletions components/zcash_note_encryption/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,15 @@ users with their own existing types can similarly implement the trait themselves
[`zcash_primitives`]: https://crates.io/crates/zcash_primitives
[`orchard`]: https://crates.io/crates/orchard

## Cross-Workspace Dependency Cycle

There is a cross-workspace cyclic dependency: `zcash_client_backend` (local)
depends on [`orchard`](https://github.com/zcash/orchard) which depends on this crate
`zcash_note_encryption`. For this reason `zcash_client_backend` does not depend on the local
`zcash_note_encryption` crate, but rather the published crate release.

This issue is tracked in [#768](https://github.com/zcash/librustzcash/issues/768).

## License

Licensed under either of
Expand Down
1 change: 1 addition & 0 deletions zcash_client_backend/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ development = ["zcash_proofs"]
incrementalmerkletree = { version = "0.4", features = ["legacy-api"] }
zcash_address = { version = "0.3", path = "../components/zcash_address" }
zcash_encoding = { version = "0.2", path = "../components/zcash_encoding" }
# Note: this is not the local workspace crate! See `components/zcash_note_encryption/README.md` for rationale:
zcash_note_encryption = "0.4"
zcash_primitives = { version = "0.12", path = "../zcash_primitives", default-features = false }

Expand Down
1 change: 1 addition & 0 deletions zcash_client_sqlite/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ proptest = "1.0.0"
rand_core = "0.6"
regex = "1.4"
tempfile = "3.5.0"
# Note: this is not the local workspace crate! See `components/zcash_note_encryption/README.md` for rationale:
zcash_note_encryption = "0.4"
zcash_proofs = { version = "0.12", path = "../zcash_proofs" }
zcash_primitives = { version = "0.12", path = "../zcash_primitives", features = ["test-dependencies"] }
Expand Down

0 comments on commit 3988578

Please sign in to comment.