Skip to content

Commit

Permalink
fix: generic Honk dependencies (AztecProtocol#4239)
Browse files Browse the repository at this point in the history
The barretenberg_module `honk`, which actually only contains a handful
of utilities used by the various flavors `ultra_honk`, `eccvm`,
`translator_vm` and `vm` 🤦 , was not being correctly reflected
in the bb module structure. I fix this by removing a whitespace
character in the CMakeLists.txt file. While I'm at it I reduce its
dependencies to the minimal set.

![reduced](https://github.com/AztecProtocol/aztec-packages/assets/26756572/fa3efd9b-1a29-4de3-8528-65426fe97333)
  • Loading branch information
codygunton authored Jan 29, 2024
1 parent 25a4bc4 commit 382dfbe
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion barretenberg/cpp/src/barretenberg/honk/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
barretenberg_module(honk proof_system commitment_schemes sumcheck)
barretenberg_module(honk polynomials)
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
#pragma once
#include "barretenberg/common/ref_vector.hpp"
#include "barretenberg/plonk/proof_system/proving_key/proving_key.hpp"
#include "barretenberg/polynomials/polynomial.hpp"
#include <typeinfo>

Expand Down

0 comments on commit 382dfbe

Please sign in to comment.