Skip to content

feat: re-land barretenberg-rs download functionality with fixes#20128

Merged
johnathan79717 merged 1 commit intomerge-train/barretenbergfrom
jh/reland-barretenberg-rs-download
Feb 3, 2026
Merged

feat: re-land barretenberg-rs download functionality with fixes#20128
johnathan79717 merged 1 commit intomerge-train/barretenbergfrom
jh/reland-barretenberg-rs-download

Conversation

@johnathan79717
Copy link
Contributor

Summary

Re-lands the download functionality from #20105 with the following fixes:

  • Check for all 0.x.x development versions, not just 0.0.1
  • Set BB_LIB_DIR in build function (ffi feature is on by default)

The build.rs now properly detects development versions and requires either
BARRETENBERG_VERSION to be set to a released version, or BB_LIB_DIR to
point to a local build.

Test plan

  • CI will verify the build and tests pass

@johnathan79717 johnathan79717 added the ci-full Run all master checks. label Feb 3, 2026
@johnathan79717 johnathan79717 force-pushed the jh/reland-barretenberg-rs-download branch from 7915627 to ce78465 Compare February 3, 2026 11:46
@AztecBot
Copy link
Collaborator

AztecBot commented Feb 3, 2026

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/f6d9fe131285d43c�f6d9fe131285d43c8;;�): cd yarn-project/kv-store && yarn test (42s) (code: 1) (\033Jonathan Hao\033: feat: re-land barretenberg-rs download functionality with fixes)

@@ -0,0 +1,4801 @@
// SPDX-License-Identifier: Apache-2.0
pragma solidity ^0.8.27;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.

Re-lands the download functionality from #20105 with the following fixes:
- Check for all 0.x.x development versions, not just 0.0.1
- Set BB_LIB_DIR in build function (ffi feature is on by default)

The build.rs now properly detects development versions and requires either
BARRETENBERG_VERSION to be set to a released version, or BB_LIB_DIR to
point to a local build.
@johnathan79717 johnathan79717 force-pushed the jh/reland-barretenberg-rs-download branch from ce78465 to 4085492 Compare February 3, 2026 12:24
@johnathan79717 johnathan79717 merged commit 383e708 into merge-train/barretenberg Feb 3, 2026
7 checks passed
@johnathan79717 johnathan79717 deleted the jh/reland-barretenberg-rs-download branch February 3, 2026 12:31
github-merge-queue bot pushed a commit that referenced this pull request Feb 3, 2026
BEGIN_COMMIT_OVERRIDE
cherchore: graph tooling audit 0 (#19691)
chore: opt sol scope renaming (#20123)
feat: add download functionality to barretenberg-rs build.rs (#20105)
fix: set BB_LIB_DIR in build function (ffi feature is on by default)
Revert barretenberg-rs download functionality
feat: re-land barretenberg-rs download functionality with fixes (#20128)
fix: skip FFI feature in PipeBackend tests
fix: skip FFI feature in PipeBackend tests
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-full Run all master checks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants