Skip to content

Commit

Permalink
Merge pull request #85 from anoma/tomas/no-js
Browse files Browse the repository at this point in the history
masp_proofs: do not enable getrandom's "js" feature by default
  • Loading branch information
tzemanovic committed Jun 25, 2024
2 parents 4ede1c4 + bfca963 commit 71b36be
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ jobs:
- run: cargo fetch
- name: Build masp_proofs for target
working-directory: ./masp_proofs
run: cargo build --verbose --no-default-features --target ${{ matrix.target }}
run: cargo build --verbose --no-default-features --target ${{ matrix.target }} --features js


bitrot:
Expand Down
3 changes: 2 additions & 1 deletion masp_proofs/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ blake2b_simd = "1"
directories = { version = "4", optional = true }
# This pulls the reddsa library that contains hardcoded personalizations and basepoints. If you need to work on a new project consider forking `reddsa`, modifying it to your needs and patching it in the root manifest
redjubjub = "0.7"
getrandom = { version = "0.2", features = ["js"] }
getrandom = { version = "0.2" }
itertools = "0.11"

[dev-dependencies]
Expand All @@ -53,6 +53,7 @@ local-prover = ["directories"]
multicore = ["bellman/multicore"]
embed-verifying-key = []
benchmarks = []
js = ["getrandom/js"]

[lib]
bench = false
Expand Down

0 comments on commit 71b36be

Please sign in to comment.