Skip to content

Commit

Permalink
fix: bootstrap bbjs. (#3337)
Browse files Browse the repository at this point in the history
As title.
Removing bootstrapping boxes as:
* It doesn't work (think it needs to be done after the yarn build), and
it doesn't fail due to missing `set -e` flags.
* It isn't needed? We only need to bootstrap what's needed to get a
working dev env for running tests.
  • Loading branch information
charlielye authored Nov 17, 2023
1 parent dd8ea9c commit 06aedcb
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 6 deletions.
9 changes: 6 additions & 3 deletions barretenberg/bootstrap.sh
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
#!/bin/bash
set -eu

# Navigate to script folder
cd "$(dirname "$0")"

(cd cpp && ./bootstrap.sh)
cd ts
yarn build
npm link
(cd ts && yarn install --immutable && yarn build && npm link)
4 changes: 2 additions & 2 deletions bootstrap.sh
Original file line number Diff line number Diff line change
Expand Up @@ -70,9 +70,9 @@ if [[ -f .bootstrapped && $(cat .bootstrapped) -eq "$VERSION" ]]; then
(cd circuits/cpp && cmake --build --preset wasm -j --target aztec3-circuits.wasm)
else
# Heavy bootstrap.
barretenberg/cpp/bootstrap.sh
barretenberg/bootstrap.sh
circuits/cpp/bootstrap.sh
yarn-project/bootstrap.sh

echo $VERSION > .bootstrapped
fi
fi
1 change: 0 additions & 1 deletion yarn-project/bootstrap.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ yarn workspace @aztec/circuits.js remake-bindings
yarn workspace @aztec/circuits.js remake-constants

(cd noir-contracts && ./bootstrap.sh)
(cd boxes && ./bootstrap.sh)
(cd .. && l1-contracts/bootstrap.sh)

# We do not need to build individual packages, yarn build will build the root tsconfig.json
Expand Down

0 comments on commit 06aedcb

Please sign in to comment.