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

Modularizing #1428

Closed
wants to merge 3 commits into from
Closed

Modularizing #1428

wants to merge 3 commits into from

Conversation

jbenet
Copy link
Member

@jbenet jbenet commented Jun 27, 2015

This PR modularizes several pieces of the ipfs codebase to make {reading, understanding, testing, making other tools} much easier.

Using godeps still becuase gx has some bugs.

Sorry, something went wrong.

https://github.com/ipfs/go-blocks

License: MIT
Signed-off-by: Juan Batiz-Benet <juan@benet.ai>
@jbenet jbenet added the status/in-progress In progress label Jun 27, 2015
jbenet added 2 commits June 27, 2015 07:20
License: MIT
Signed-off-by: Juan Batiz-Benet <juan@benet.ai>
License: MIT
Signed-off-by: Juan Batiz-Benet <juan@benet.ai>
@whyrusleeping
Copy link
Member

i'd prefer this type of stuff wait until go1.5, having to type github.com/ipfs/go-ipfs/Godeps/_workspace/src/github.com/ipfs/go-merkledag every time i want to use the merkledag package is very inconvenient...

@jbenet
Copy link
Member Author

jbenet commented Jun 29, 2015

having to type github.com/ipfs/go-ipfs/Godeps/_workspace/src/github.com/ipfs/go-merkledag every time i want to use the merkledag package is very inconvenient...

type github.com/ipfs/go-merkledag + make vendor, our favorite friend :)

@whyrusleeping whyrusleeping added sprint and removed status/in-progress In progress labels Jul 2, 2015
@whyrusleeping
Copy link
Member

closing, old PR cleanup time.

@jbenet jbenet removed the sprint label Oct 18, 2015
@Kubuxu Kubuxu deleted the modularizing branch February 27, 2017 20:40
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.

None yet

2 participants