Skip to content

Align macOS filename with others in checksum file #158

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.

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

Merged
merged 1 commit into from
Sep 1, 2021
Merged

Align macOS filename with others in checksum file #158

merged 1 commit into from
Sep 1, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Sep 1, 2021

A file containing a list of checksums for the build archives is generated by the system used by the "Release Go"
template. The macOS build archive is modified later in the process as a result of the repackaging following notarization.
For this reason, the checksum file must be updated. The sha256sum command used to generate the original file separates
the checksum from the filename with two spaces. Previously, the command that updates the macOS checksum only added a
single space, which caused the formatting of the file to be inconsistent:

d5dccece4253b8c4c7e86b80f77433b3f3363ee89dcf11fd91d4638bbc66b61e  bhy_0.0.1_Linux_32bit.tar.gz
15557602d6aa6fd0a64b88393b7bd06fde35d6855a51c6b6a56a89ebd7d55153  bhy_0.0.1_Linux_64bit.tar.gz
d7564496bfbb471a79365a838ce6c8a2bc41285b1b687d58747308c78c831332 bhy_0.0.1_macOS_64bit.tar.gz

A file containing a list of checksums for the build archives is generated by the system used by the "Release Go"
template. The macOS build archive is modified later in the process as a result of the repackaging following notarization.
For this reason, the checksum file must be updated. The `sha256sum` command used to generate the original file separates
the checksum from the filename with two spaces. Previously, the command that updates the macOS checksum only added a
single space, which caused the formatting of the file to be inconsistent:

```
d5dccece4253b8c4c7e86b80f77433b3f3363ee89dcf11fd91d4638bbc66b61e  bhy_0.0.1_Linux_32bit.tar.gz
15557602d6aa6fd0a64b88393b7bd06fde35d6855a51c6b6a56a89ebd7d55153  bhy_0.0.1_Linux_64bit.tar.gz
d7564496bfbb471a79365a838ce6c8a2bc41285b1b687d58747308c78c831332 bhy_0.0.1_macOS_64bit.tar.gz
```
@per1234 per1234 added type: imperfection Perceived defect in any part of project topic: code Related to content of the project itself labels Sep 1, 2021
@per1234 per1234 requested a review from umbynos September 1, 2021 06:33
@per1234 per1234 merged commit b521f45 into arduino:main Sep 1, 2021
@per1234 per1234 deleted the checksum-formatting branch September 1, 2021 07:41
@per1234 per1234 self-assigned this Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants