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

use gx vendored packages instead of godeps ones #2319

Merged
merged 5 commits into from
Feb 13, 2016

Conversation

whyrusleeping
Copy link
Member

This PR removes a few deps from godeps (one at a time, mass dep changes are bad).

It also uses the gx vendored go-ipfs-util instead of the in-repo 'util' package, a few subpackages of util that werent included in go-ipfs-util have been moved to thirdparty.

NOTE:
All packages in thirdparty are prime canidates for being extracted from go-ipfs into their own separate gx packages. I'm leaving them there so that anyone who wants to try out gx has a good clear target of what to do.

@whyrusleeping whyrusleeping force-pushed the cleanup/gx-godeps branch 2 times, most recently from 9946ed7 to b524b8b Compare February 10, 2016 03:27
@whyrusleeping
Copy link
Member Author

saw this test failure: https://travis-ci.org/ipfs/go-ipfs/jobs/108191449 and was sad because i was sure i fixed it. Then i realized i hadnt rebased this over that fix. 😓

License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
For the rest of the packages in util, move them to thirdparty
and update the references. util is gone!

License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
whyrusleeping added a commit that referenced this pull request Feb 13, 2016
use gx vendored packages instead of godeps ones
@whyrusleeping whyrusleeping merged commit 6b76e2a into master Feb 13, 2016
@whyrusleeping whyrusleeping deleted the cleanup/gx-godeps branch February 13, 2016 22:46
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.

1 participant