Skip to content
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

Check downloaded gcc/clang checksums #1367

Merged
merged 3 commits into from
Feb 9, 2025
Merged

Check downloaded gcc/clang checksums #1367

merged 3 commits into from
Feb 9, 2025

Conversation

bjia56
Copy link
Contributor

@bjia56 bjia56 commented Feb 9, 2025

Check sha256 checksums of the downloaded gcc/clang toolchains. This allows us to extend trust to the external toolchains when building from source.

@github-actions github-actions bot added the tool label Feb 9, 2025
jart
jart approved these changes Feb 9, 2025
View reviewed changes
Copy link
Owner

@jart jart left a comment

Choose a reason for hiding this comment

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

Thanks! You could also probably use .cosmocc/3.9.2/bin/sha256sum.ape but this is fine.

@jart jart merged commit 0e557d0 into jart:master Feb 9, 2025
6 checks passed
@jart
Copy link
Owner

jart commented Feb 9, 2025

Commit message modified to follow 72-column rule. See https://youtu.be/Y65FRxE7uMc for further details.

@bjia56 bjia56 deleted the check-compilers-checksum branch February 9, 2025 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jart jart jart approved these changes

Assignees
No one assigned
Labels
tool
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bjia56 @jart