Skip to content

Commit

Permalink
Don't unify "real" and dev dependencies
Browse files Browse the repository at this point in the history
This commit moves the library and `binaries` targets to `resolver = "2"`
so that `dev-dependencies` don't get pulled in when building non-test
configurations of the `prio` library. Up until now, the crate releases
were enabling feature `test-vector`, unnecessarily including the
`test_vector` module and its dependencies. See [1] for details on
dependency resolution with the new resolver.

[1]: rust-lang/cargo#4866
  • Loading branch information
tgeoghegan committed Jul 7, 2022
1 parent aa19100 commit 8cbf27c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ description = "Implementation of the Prio aggregation system core: https://crypt
license = "MPL-2.0"
repository = "https://github.com/divviup/libprio-rs"
rust-version = "1.58"
resolver = "2"

[dependencies]
aes = "0.8.1"
Expand Down Expand Up @@ -46,8 +47,7 @@ test-vector = ["rand", "serde_json"]
multithreaded = ["rayon"]

[workspace]
members = ["binaries"]
default-members = ["binaries", "."]
members = [".", "binaries"]

[[bench]]
name = "speed_tests"
Expand Down
1 change: 1 addition & 0 deletions binaries/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ edition = "2018"
description = "Prio utilities"
license = "MPL-2.0"
repository = "https://github.com/divviup/libprio-rs"
resolver = "2"

[dependencies]
base64 = "0.13.0"
Expand Down

0 comments on commit 8cbf27c

Please sign in to comment.