Skip to content

Fix sha256 for cvc5_mac#180

Merged
phiwuu merged 1 commit intobmw-software-engineering:mainfrom
vishwajitdandage:main
Apr 27, 2026
Merged

Fix sha256 for cvc5_mac#180
phiwuu merged 1 commit intobmw-software-engineering:mainfrom
vishwajitdandage:main

Conversation

@vishwajitdandage
Copy link
Copy Markdown
Contributor


Issue: Incorrect CVC5 1.3.2 macOS ARM64 SHA256 Checksum in MODULE.bazel

Description:
The TRLC MODULE.bazel contains an incorrect SHA256 checksum for the CVC5 1.3.2 macOS ARM64 static GPL binary.

Current (incorrect) value:

sha256 = "9b00ae44d51903cd830e281c97066d7e4c0a57f1aa6c5b275435d3016427be64"

Expected (correct) value:

sha256 = "6c63ba3bd174d026be161be75f64df8d200c8a284012c49e91807f3ede5afc44"

Verification:
The correct checksum can be verified by downloading the file directly:

wget https://github.com/cvc5/cvc5/releases/download/cvc5-1.3.2/cvc5-macOS-arm64-static-gpl.zip
shasum -a 256 cvc5-macOS-arm64-static-gpl.zip
# Output: 6c63ba3bd174d026be161be75f64df8d200c8a284012c49e91807f3ede5afc44

Bazel Impact:

  • Checksum Validation Failure: Bazel 8+ strict downloader validation will reject the download with: checksum mismatch: expected 9b00ae44..., got 6c63ba3bd...
  • CI/CD Breakage: Prevents bazel mod tidy and bazel test from succeeding.
  • Workaround Required: Users must apply local patches to override the incorrect checksum, creating maintenance burden

Impact:
Users attempting to use TRLC with Bazel's downloader/http_archive will face checksum validation failures, requiring local patches to work around the issue.

Fix:
Update the SHA256 value in MODULE.bazel for the cvc5_mac http_archive rule to use the correct checksum.


@vishwajitdandage vishwajitdandage requested a review from a team as a code owner April 24, 2026 07:21
@phiwuu
Copy link
Copy Markdown
Member

phiwuu commented Apr 24, 2026

Thank you for your PR, we'll investigate it.

@hoe-jo hoe-jo self-requested a review April 27, 2026 06:33
Copy link
Copy Markdown
Contributor

@hoe-jo hoe-jo left a comment

Choose a reason for hiding this comment

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

Correct SHA256, the one in the repo was for x86 not arm

@phiwuu phiwuu merged commit 44f23de into bmw-software-engineering:main Apr 27, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants