Skip to content

Support GOARM and windows .exe in binary target#1857

Merged
thaJeztah merged 1 commit intodocker:masterfrom
cpuguy83:allow_better_cross_support
May 14, 2019
Merged

Support GOARM and windows .exe in binary target#1857
thaJeztah merged 1 commit intodocker:masterfrom
cpuguy83:allow_better_cross_support

Conversation

@cpuguy83
Copy link
Copy Markdown
Collaborator

@cpuguy83 cpuguy83 commented May 2, 2019

This just makes it easier to build a targeted binary for the
goos/goach/goarm version.

This of course will not work for all cases but is nice to get things
going.
Specifically cross-compiling pkcs for yubikey support requires some
extra work which is not tackled here.

This just makes it easier to build a targeted binary for the
goos/goach/goarm version.

This of course will not work for all cases but is nice to get things
going.
Specifically cross-compiling pkcs for yubikey support requires some
extra work whichis not tackled here.

Signed-off-by: Brian Goff <[email protected]>
@cpuguy83 cpuguy83 force-pushed the allow_better_cross_support branch from 6883f55 to 15130e3 Compare May 6, 2019 17:43
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1857 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1857   +/-   ##
=======================================
  Coverage   56.75%   56.75%           
=======================================
  Files         309      309           
  Lines       21658    21658           
=======================================
  Hits        12292    12292           
  Misses       8469     8469           
  Partials      897      897

@cpuguy83 cpuguy83 requested a review from vdemeester May 7, 2019 22:52
@cpuguy83
Copy link
Copy Markdown
Collaborator Author

Ping

Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

ping @tonistiigi @tiborvass PTAL

Copy link
Copy Markdown
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐯

@thaJeztah thaJeztah merged commit 6402c0a into docker:master May 14, 2019
@GordonTheTurtle GordonTheTurtle added this to the 19.09.0 milestone May 14, 2019
@cpuguy83 cpuguy83 deleted the allow_better_cross_support branch June 5, 2019 21:12
thaJeztah added a commit that referenced this pull request Jun 6, 2019
[19.03] Backport #1857: Support GOARM and windows .exe in binary target
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.

5 participants