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

chore: update deps #8128

Merged
merged 2 commits into from
May 12, 2021
Merged

chore: update deps #8128

merged 2 commits into from
May 12, 2021

Conversation

aschmahmann
Copy link
Contributor

No description provided.

storeutil.LoaderForBlockstore(bs),
storeutil.StorerForBlockstore(bs),
storeutil.LinkSystemForBlockstore(bs),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Required by go-graphsync upgrade

go.mod Outdated
Comment on lines 45 to 51
github.com/ipfs/go-ipns v0.0.2
github.com/ipfs/go-ipns v0.0.3-0.20210511191739-3deb032d2893
github.com/ipfs/go-log v1.0.5
github.com/ipfs/go-merkledag v0.3.2
github.com/ipfs/go-metrics-interface v0.0.1
github.com/ipfs/go-metrics-prometheus v0.0.2
github.com/ipfs/go-mfs v0.1.2
github.com/ipfs/go-namesys v0.2.0
github.com/ipfs/go-namesys v0.2.1-0.20210511192903-e958ffb989d2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will update to released version after tests pass here

echo foo | test_must_fail ipfs add --hash murmur3-128 2>add_out
echo foo | test_must_fail ipfs add --hash shake-128 2>add_out
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a consequence of updating go-multihash which has removed the exposure of the murmur3 hash function due to some issues multiformats/go-multihash#135

@aschmahmann aschmahmann requested a review from Stebalien May 12, 2021 19:31
@aschmahmann aschmahmann merged commit 56f61d5 into master May 12, 2021
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