Skip to content

Commit

Permalink
fix: update protogalaxy cmake dependencies (#5066)
Browse files Browse the repository at this point in the history
Closes AztecProtocol/barretenberg#854.

We previously had to make ultra_honk dependent on protogalaxy because we
had the ultra composer in the ultra_honk folder, and that composer had
protogalaxy related functions. Now, with the composer dead, we can have
the correct dependency, which is going the other way.

Old graph:

![barretenberg_modules](https://github.com/AztecProtocol/aztec-packages/assets/42556996/99866e8d-eaa8-45de-855e-d2c886c4f0a0)

New graph:

![barretenberg_modules](https://github.com/AztecProtocol/aztec-packages/assets/42556996/210301b2-abb9-469b-a6b3-c50832f318fa)

No benchmarking update since this should not affect it.
  • Loading branch information
lucasxia01 authored Mar 7, 2024
1 parent 0e44247 commit 507c374
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -1 +1 @@
barretenberg_module(protogalaxy honk flavor relations sumcheck)
barretenberg_module(protogalaxy ultra_honk flavor relations sumcheck)
Original file line number Diff line number Diff line change
@@ -1 +1 @@
barretenberg_module(stdlib_recursion ecc proof_system stdlib_primitives stdlib_pedersen_commitment stdlib_blake3s ultra_honk eccvm translator_vm stdlib_poseidon2 circuit_checker)
barretenberg_module(stdlib_recursion ecc proof_system stdlib_primitives stdlib_pedersen_commitment stdlib_blake3s protogalaxy eccvm translator_vm stdlib_poseidon2 circuit_checker)
Original file line number Diff line number Diff line change
@@ -1 +1 @@
barretenberg_module(ultra_honk honk sumcheck protogalaxy)
barretenberg_module(ultra_honk honk sumcheck)

0 comments on commit 507c374

Please sign in to comment.