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

Add blake3 hash and sharness tests #147

Merged
merged 2 commits into from
Sep 20, 2021

Conversation

jbouwman
Copy link
Contributor

This PR registers the version of BLAKE3 hashing from https://github.com/lukechampine/blake3 which now seems to have vector optimization.

Closes #121

go.sum Outdated
Comment on lines 23 to 24
lukechampine.com/blake3 v1.1.5 h1:hsACfxWvLdGmjYbWGrumQIphOvO+ZruZehWtgd2fxoM=
lukechampine.com/blake3 v1.1.5/go.mod h1:hE8RpzdO8ttZ7446CXEwDP1eu2V4z7stv0Urj1El20g=
Copy link
Member

Choose a reason for hiding this comment

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

💭 what happens when domain is not renewed? :trollface:

Choose a reason for hiding this comment

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

I expect I'll continue to own lukechampine.com until I die. After that (if people are still using my code), they can rehost the source somewhere else. (lukechampine.com/foo just redirects to my github.com/lukechampine/foo)

@Stebalien
Copy link
Member

lukechampine/blake3#13

But the cpuid version doesn't really matter.

@lukechampine
Copy link

I've tagged v1.1.6 with cpuid v2 if you want to use that. 👍🏻

@Stebalien
Copy link
Member

Thanks!

@aschmahmann aschmahmann merged commit 82c18ac into multiformats:master Sep 20, 2021
@aschmahmann aschmahmann mentioned this pull request Dec 1, 2021
80 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.

[enhancement] adding BLAKE3 support
5 participants