-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Update docker dependency for pkg/term #59
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
Conversation
mlaventure
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
dnephin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Quickly hold off on this, i need to make one more update to the non-cgo version of the term package to fix static builds |
|
moby/moby#33128 merged! 😄 🎉 |
b59035d to
bccd8b4
Compare
|
Ok, updated this to the latest docker after the update pr is merged. This is ready for review and merge. Sorry about the wait |
We need the `pkg/term` dep from docker with OPOST added to the MakeRaw call in order for the new runc 1.0 integration to work properly. Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
bccd8b4 to
56ea98f
Compare
There seems to be an issue with vendoring |
|
@mlaventure what??? ;) updated |
mlaventure
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Update deprecated.md for removal of --email flag
We need the
pkg/termdep from docker with OPOST added to the MakeRawcall in order for the new runc 1.0 integration to work properly.
Signed-off-by: Michael Crosby crosbymichael@gmail.com