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

[Carry] Fix a panic where RUN [] would be supplied #10108

Merged
merged 2 commits into from
Jan 15, 2015

Conversation

tiborvass
Copy link
Contributor

Closes #9989

/cc @erikh @duglin

Erik Hollensbe and others added 2 commits January 14, 2015 16:55
Docker-DCO-1.1-Signed-off-by: Erik Hollensbe <github@hollensbe.org> (github: erikh)
Signed-off-by: Tibor Vass <teabee89@gmail.com>
@vieux
Copy link
Contributor

vieux commented Jan 15, 2015

LGTM

1 similar comment
@crosbymichael
Copy link
Contributor

LGTM

@tiborvass tiborvass added this to the 1.5.0 milestone Jan 15, 2015
@jessfraz
Copy link
Contributor

whoa you all not going to wait for drone

@crosbymichael
Copy link
Contributor

In @tiborvass I Trust

@tiborvass
Copy link
Contributor Author

@crosbymichael famous last words :D

@jessfraz
Copy link
Contributor

LGTM

jessfraz pushed a commit that referenced this pull request Jan 15, 2015
[Carry] Fix a panic where RUN [] would be supplied
@jessfraz jessfraz merged commit eaecf74 into moby:master Jan 15, 2015
@jessfraz
Copy link
Contributor

would have been funny if it failed on random racy test

@tiborvass tiborvass deleted the carry-9989 branch January 15, 2015 01:13
@tiborvass
Copy link
Contributor Author

@jfrazelle I had a 50/50 chance :)

@erikh
Copy link
Contributor

erikh commented Jan 15, 2015

Thank you so much @tiborvass!

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

5 participants