Skip to content

[19.03] Backport #1857: Support GOARM and windows .exe in binary target#1927

Merged
thaJeztah merged 1 commit intodocker:19.03from
cpuguy83:backport_goarm
Jun 6, 2019
Merged

[19.03] Backport #1857: Support GOARM and windows .exe in binary target#1927
thaJeztah merged 1 commit intodocker:19.03from
cpuguy83:backport_goarm

Conversation

@cpuguy83
Copy link
Copy Markdown
Collaborator

@cpuguy83 cpuguy83 commented Jun 5, 2019

Backports #1857

Figured lets go ahead and get this on the 19.03 tree.


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]
(cherry picked from commit 15130e3)
Signed-off-by: Brian Goff [email protected]

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]>
(cherry picked from commit 15130e3)
Signed-off-by: Brian Goff <[email protected]>
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1927 into 19.03 will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            19.03    #1927   +/-   ##
=======================================
  Coverage   56.69%   56.69%           
=======================================
  Files         310      310           
  Lines       21778    21778           
=======================================
  Hits        12346    12346           
  Misses       8519     8519           
  Partials      913      913

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

@thaJeztah
Copy link
Copy Markdown
Member

all green; I'll get this one in

@thaJeztah thaJeztah merged commit a63faeb into docker:19.03 Jun 6, 2019
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.

4 participants