fix: Update nargo and bb installations #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

Savio-Sou wants to merge 1 commit into master from ss/update-installations

Contributor

Savio-Sou commented Aug 25, 2025

Description

Problem*

For nargo installation:

The version of noirup script used is a little old (v0.1.3 vs v0.1.4)
The version of nargo used is quite old (v0.36.0 vs v1.0.0-beta.11)
The version of nargo used is hard-coded at different places

For bb installation:

The downloading of bbup relies on bbup the NPM package rather than the raw GitHub source, which introduces additional supply attack chain vectors and is prone to falling out of date

Summary*

For nargo installation:

Update to use v0.1.4 noirup
Update to use Noir v1.0.0-beta.11
Add and use an environmental variable for targetted Noir version

For bb installation:

Switch to using https://raw.githubusercontent.com/AztecProtocol/aztec-packages/refs/heads/next/barretenberg/bbup/install for installing bbup instead

PR Checklist*

I have tested the changes locally.
I have formatted the changes with Prettier and/or cargo fmt on default settings.


          Update nargo and bb installations

738402a

Savio-Sou requested a review from jialinli98

August 25, 2025 15:19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet