Skip to content
This repository has been archived by the owner on Apr 16, 2020. It is now read-only.

Review ipfs-pack draft proposal #99

Closed
flyingzumwalt opened this issue Jan 15, 2017 · 7 comments
Closed

Review ipfs-pack draft proposal #99

flyingzumwalt opened this issue Jan 15, 2017 · 7 comments
Assignees

Comments

@flyingzumwalt
Copy link
Contributor

flyingzumwalt commented Jan 15, 2017

Review Issue: Proposing some tooling for datasets (ipfs-pack and stuff)

Implementing a basic version of pack should be relatively simple and would be quite useful because it allows a dataset to be a freestanding content-addressed thing regardless of how you're storing it. Among other things, this allows us to simplify the filestore requirements.

Related Stories: #98

@kevina
Copy link

kevina commented Jan 16, 2017

What exactly are the "filestore requirements." that seams very odd that now we are working on this, considering I already have a very functional implementation?

@flyingzumwalt
Copy link
Contributor Author

@kevina I recommend posing this question on Issue #85 . My understanding is that @whyrusleeping and @jbenet think the existing implementation is too complicated. They plan to review the filestore implementation and are considering the "Concise Filestore spec" linked to from that issue.

@kevina
Copy link

kevina commented Jan 16, 2017

@flyingzumwalt okay. the question wasn't really directed at you.

@jbenet
Copy link
Contributor

jbenet commented Jan 18, 2017

@kevina context: ipfs/team-mgmt#309 (comment)

@jbenet
Copy link
Contributor

jbenet commented Jan 19, 2017

I need to update the ipfs-pack spec for the work we're doing this sprint.

@kevina
Copy link

kevina commented Jan 19, 2017

There is a good chance I can help with ipfs-pack but I want to first work on any filestore specific issues so I am waiting for @jbenet notes on my code and/or meeting before I commit any time to this sprint.

@flyingzumwalt
Copy link
Contributor Author

Results of this review: ipfs/ipfs-pack repo with new ipfs-pack spec

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants