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

Build ipfs cmd inside docker container #2213

Merged
merged 1 commit into from Jan 17, 2016
Merged

Build ipfs cmd inside docker container #2213

merged 1 commit into from Jan 17, 2016

Conversation

Luzifer
Copy link
Member

@Luzifer Luzifer commented Jan 17, 2016

@jbenet PTAL

Please note: Merging this makes #2169 obsolete (close that one after merging this)!

License: MIT
Signed-off-by: Knut Ahlers <knut@ahlers.me>
@jbenet
Copy link
Member

jbenet commented Jan 17, 2016

This is great! thanks @Luzifer

@chriscool do the docker tests we have test our Dockerfile? may be worth having a test for it if we don't alrady.

jbenet added a commit that referenced this pull request Jan 17, 2016
Build ipfs cmd inside docker container
@jbenet jbenet merged commit 89a6f01 into ipfs:master Jan 17, 2016
@jbenet jbenet deleted the direct_build branch January 17, 2016 20:42
@chriscool
Copy link
Contributor

@ jbenet yeah PR #2064 added https://github.com/ipfs/go-ipfs/blob/master/test/sharness/t0300-docker-image.sh where we build a Docker image from our Dockerfile and run a simple command on it.

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

3 participants