change Dockerfile order to create VOLUME under user ipfs

[ChristianKniep]

License: MIT
Signed-off-by: Christian Kniep christian@qnib.org

Changed the commit message...

[whyrusleeping]

this seems correct to me, @jbenet @Luzifer ?

[Luzifer]

Hadn't time to test yet but either it makes things better or it doesn't change things. So no negative results expected. LGTM

[jbenet]

would be good to have a sharness test that tests the Dockerfile

[whyrusleeping]

test it by running a docker image with the file?

[jbenet]

yep!

On Friday, October 9, 2015, Jeromy Johnson notifications@github.com wrote:

test it by running a docker image with the file?

—
Reply to this email directly or view it on GitHub
#1806 (comment).

[whyrusleeping]

are we saying that a test on the dockerfile is a hard requirement for this PR?

[jbenet]

Well We should confirm it works, and that it be broken again. If not now, when?

[whyrusleeping]

just clarifying so we know what tasks need to be done before this PR can merge.

[jbenet]

@ChristianKniep are you familiar with our sharness testing and would you have time to do this?

else, is anyone else able to?

[ghost]

continued in #2064