-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
vendoring #28
Comments
Yeah we need to vendor everything.— On Fri, Aug 15, 2014 at 10:11 PM, Patrick Pfeiffer
|
best strategies? the way docker does it? |
im a complete go-noob, without any experience. if you want me to i can put all the deps into i am currently looking into all the tools around golang, the dependency-management seems to be kind of rough (comming from nodejs&npm). there are tools to help with vendoring but none of them is ideal. |
I feel like we should have an ipfs github organization for any vendoring we do. |
Am not too keen on premature orgs -- they add a whole bunch of hassle up front. once we have many repos, sure. (For vendoring, we don't need repos. We should be vendoring like Docker + Camlistore do -- cloning into the repo). At least until we can host the code / package manager on IPFS. :) |
i cant wait to host all the things in ipfs, like everything hahaha also i think vendoring like camli and docker is the best (no git-submodules) recently i saw another tool that might not be so bad, its used in kubernetes: https://github.com/GoogleCloudPlatform/kubernetes/tree/master/Godeps |
I'd like suggest considering Godeps, too. If all you need is locking your dependencies to specific commits/tags/versions, Godep is a very nice solution. I think docker and camlistore chose their ways because no other mature solution was available when they started. |
my issue with godeps is that its another external tool we have to be dependant on, vendoring the code is seamless and from the users perspective, no extra steps are required. |
I also want to mention that vendoring gives us the power to update our dependencies without having to push any new commits to ipfs, having 400 commits saying "Updating godeps" seems like a messy solution to me |
the reason for vendoring is to update dependencies by commit, isnt it? you want to be able to checkout a specific commit of ipfs and be sure to get exactly the same binary with |
Yeah, we want it to be committed. Let's vendor the way docker + camlistore do it for now. |
Ah, I get it now. |
Please see: #35 |
Shall we close this issue? |
yep |
currently
go get github.com/jbenet/go-ipfs/cmd/ipfs
doesnt work because of breaking changes in gogoprotobuf, so maybe put deps in a vendor-directory?https://botbot.me/freenode/ipfs/2014-08-15/?msg=19828336&page=1
The text was updated successfully, but these errors were encountered: