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

Update deps to match go-ipfs-master #805

Merged
merged 0 commits into from
Aug 28, 2018
Merged

Update deps to match go-ipfs-master #805

merged 0 commits into from
Aug 28, 2018

Conversation

dignifiedquire
Copy link
Contributor

I wanted to update go-floodsub, to prepare for integrating gossipsub. It needed a few dependency changes to make it work. This brings us in line with the dependencies used in go-ipfs master as of this writing.

@whyrusleeping
Copy link
Member

Btw, you don't need to update to enable gossipsub

@dignifiedquire
Copy link
Contributor Author

dignifiedquire commented Aug 28, 2018

Btw, you don't need to update to enable gossipsub

I realized that halfway through :D But now we have all the new cool stuff

@whyrusleeping whyrusleeping merged this pull request into master Aug 28, 2018
@whyrusleeping whyrusleeping deleted the deps/update branch August 28, 2018 22:25
@whyrusleeping
Copy link
Member

whoooooaaa.... something was messed up severely here... Got lots of duplicate deps issues. :/

@whyrusleeping
Copy link
Member

@dignifiedquire several of the intermediate deps had the wrong versions in the tree. You can check that with gx deps dupes. Also use gx deps --tree --highlight=PACKAGENAME to see where things are off.

Best way to update deps (for now) is to use gx-workspace. I would start it with gx-workspace update start go-cid (as i think go-cid is the root of all these changes) and then just run gx-workspace update next and follow the prompts to complete the update.

cc @Stebalien

@dignifiedquire
Copy link
Contributor Author

That is failing pretty bad because of the of filecoin-project/go-ipld-cbor and the dependency through of it from go-hamt-ipld -.-

@dignifiedquire
Copy link
Contributor Author

I am giving up on this for now...

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