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.

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

fix: do not include full path to binary in .sha512 files #1108

Merged
merged 2 commits into from
Nov 8, 2024
Merged

Conversation

hsanjuan
Copy link
Contributor

@hsanjuan hsanjuan commented Nov 8, 2024

See ipfs/kubo#9323.

This is a darwin release thing because the signature process for macOS means that we recalculate sha512 and cid and we do this using an absolute path so the result includes the path to the file.

The fix is to cd to the folder and do it from there.

See ipfs/kubo#9323.

This is a darwin release thing because the signature process for macOS means
that we recalculate sha512 and cid and we do this using an absolute path so
the result includes the path to the file.

The fix is to cd to the folder and do it from there.
Copy link

github-actions bot commented Nov 8, 2024

This change produced no new differences in built artifacts.

@lidel
Copy link
Member

lidel commented Nov 8, 2024

Thanks!

I'll try to test.

Manually dispatched https://github.com/ipfs/distributions/actions/workflows/main.yml
with branch set to fix/kubo-9323 (this PR) and DIST_ROOT set to /ipfs/QmeqkkydCxmREqg2qmoA7XQSYbcgvSko5zDAhnarKFp2Ki (version without 0.32.0-rc2, so we have actual build happening) to see if produced DAG has expected values, but it did not work the way I expected.

Update: we had a bug (116dfe2), will fix and retry.

Update: 0.32.0-rc2 builds, let's wait until this CI run is done.

name did not match one used in DIST_ROOT section

(cherry picked from commit a00d71e)
Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

lgtm, https://github.com/ipfs/distributions/actions/runs/11750043494/job/32738190122#step:9:10 produced DAG with expected value:

$ ipfs cat /ipfs/Qmbu8BBcVVT6JpnQWssMwmzGaD6YQeB8XV7PtU4yp1BgNo/kubo/v0.32.0-rc2/kubo_v0.32.0-rc2_darwin-amd64.tar.gz.sha512
dccbd57d1d66e9cb95845c9b251d4b190771141451f50bb4e88710f4e4d869356a916de0f5ef14ec23f633865c88451194a2565036fac929bd2339424cde167b  kubo_v0.32.0-rc2_darwin-amd64.tar.gz

Merging so the fix is applied when we release Kubo 0.32.0 next week.

@lidel lidel changed the title Do not include full path to binary in .sha512 files on darwin fix: do not include full path to binary in .sha512 files Nov 8, 2024
@lidel lidel merged commit 28901ac into master Nov 8, 2024
5 of 10 checks passed
@lidel lidel deleted the fix/kubo-9323 branch November 8, 2024 22:52
@lidel lidel mentioned this pull request Nov 8, 2024
28 tasks
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.

2 participants