Conversation
|
I'm having a minor problem with the 2nd test that has stumped me. The test fails with I cannot work out why its complaining that it can't see the Dockerfile in that dir. |
There was a problem hiding this comment.
You can try file existence just with [ -e file ]
|
updated, my sh skills need work. |
|
Added another test, 2/3 are still failing though for reasons I can't figure out. |
|
now rebased on dotcloud/docker |
|
I managed to screw this up with the rebase, going to close and do it again cleanly |
|
Cool, no problem |
Docker-DCO-1.1-Signed-off-by: Travis Cline <travis.cline@gmail.com> (github: tmc)
Fixes moby#2224 Docker-DCO-1.1-Signed-off-by: Travis Cline <travis.cline@gmail.com> (github: tmc)
Docker-DCO-1.1-Signed-off-by: Brad Murray <wyaeld@gmail.com> (github: wyaeld)
Docker-DCO-1.1-Signed-off-by: Brad Murray <wyaeld@gmail.com> (github: wyaeld)
|
This time its the correct commits, without any mistakes in the rebase. Still 2 problems.
|
|
@wyaeld Yeah, ignoring Dockerfile doesn't make sense, I think we should put check about it in |
|
Ok, so the
|
Move environment loading to nsenter
Added 2 new integration-cli tests for the dockerignore feature