Skip to content

Remove gox, add support for s390x - #129

Merged
mlaventure merged 1 commit into
docker:masterfrom
dnephin:remove-gox
May 30, 2017
Merged

mlaventure merged 1 commit into
docker:masterfrom
dnephin:remove-gox

Conversation

@dnephin

@dnephin dnephin commented May 25, 2017

Copy link
Copy Markdown
Contributor

Fixes #31

Removes gox and scripts/build/linux-cross
Adds support for linux/s390x

cc @andrewhsu @cpuguy83

@codecov-io

codecov-io commented May 25, 2017 •

Copy link
Copy Markdown

Codecov Report

Merging #129 into master will increase coverage by 1.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
+ Coverage   44.85%   46.12%   +1.27%     
==========================================
  Files         169      161       -8     
  Lines       11376    11006     -370     
==========================================
- Hits         5103     5077      -26     
+ Misses       5981     5639     -342     
+ Partials      292      290       -2

@vdemeester vdemeester left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👼

@dnephin

dnephin commented May 30, 2017

Copy link
Copy Markdown
Contributor Author

ping, this should be an easy review

@dnephin
dnephin requested a review from mlaventure May 30, 2017 19:44
Comment thread scripts/build/cross Outdated
"$BUILDDIR/linux-cross" \
"$BUILDDIR/windows" \
"$BUILDDIR/osx" \
"GOARCH=amd64 $BUILDDIR/binary" \

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this set GOOS?

@dnephin dnephin May 30, 2017 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess right now it assumes that it's always run in a linux container, but that's not necessarily the case.

I've added GOOS=linux

Signed-off-by: Daniel Nephin <dnephin@docker.com>

@cpuguy83 cpuguy83 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mlaventure mlaventure left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mlaventure
mlaventure merged commit 6c59636 into docker:master May 30, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.06.0 milestone May 30, 2017
@dnephin
dnephin deleted the remove-gox branch June 5, 2017 22:10
@thaJeztah

Copy link
Copy Markdown
Member

trapier pushed a commit to trapier/cli that referenced this pull request Sep 30, 2019
…-19.03-c500b534e741395dd4f728f458dd0a95479de3a7

[19.03] sync to upstream 19.03 c500b53
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants