Skip to content

fix: use pinned solc for bb solidity tests#24097

Draft
AztecBot wants to merge 1 commit into
merge-train/spartanfrom
cb/fix-spartan-bb-sol-solc
Draft

fix: use pinned solc for bb solidity tests#24097
AztecBot wants to merge 1 commit into
merge-train/spartanfrom
cb/fix-spartan-bb-sol-solc

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

  • Move the barretenberg/sol solc override into [profile.default] so Foundry uses ../../l1-contracts/solc-0.8.30.
  • Prevent forge test --no-match-contract Base from falling back to SVM and fetching binaries.soliditylang.org during CI.

Verification

  • cd l1-contracts && ./bootstrap.sh download_solc
  • cd barretenberg/sol && forge config --json | jq -r '.solc' -> ../../l1-contracts/solc-0.8.30
  • cd barretenberg/sol && ./bootstrap.sh
  • cd barretenberg/sol && forge test --no-match-contract Base (168 passed, 0 failed)
  • ./bootstrap.sh ci is not a supported command in this checkout (Unknown command: ci).
  • ./bootstrap.sh ci-full-no-test-cache was attempted; the fixed bb-sol job passed, but the full local run is blocked by unrelated local environment issues: spartan needs sudo to install kubectl, and noir cannot fetch https://index.crates.io/config.json due local proxy/DNS errors.

Created by claudebox · group: slackbot

@AztecBot AztecBot added ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR. labels Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant