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

bake/bin-image-cross: Add darwin #4996

Merged
merged 3 commits into from
May 21, 2024
Merged

Conversation

vvoland
Copy link
Contributor

@vvoland vvoland commented Apr 8, 2024

So we can also have darwin binaries in the dockereng/cli-bin image.

Signed-off-by: Paweł Gronowski [email protected]

@vvoland vvoland added this to the 27.0.0 milestone Apr 8, 2024
@vvoland vvoland self-assigned this Apr 8, 2024
@codecov-commenter
Copy link

codecov-commenter commented Apr 8, 2024

Codecov Report

Merging #4996 (cb9c97d) into master (870ad7f) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head cb9c97d differs from pull request most recent head 1433df8. Consider uploading reports for the commit 1433df8 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4996      +/-   ##
==========================================
- Coverage   61.04%   61.03%   -0.01%     
==========================================
  Files         295      295              
  Lines       20648    20648              
==========================================
- Hits        12605    12603       -2     
- Misses       7147     7148       +1     
- Partials      896      897       +1     

@vvoland vvoland force-pushed the cli-bin-darwin branch 2 times, most recently from a08a097 to 7e95682 Compare April 10, 2024 14:28
Cross compiling CGO to Darwin requires an Apple SDK.

Signed-off-by: Paweł Gronowski <[email protected]>
Signed-off-by: Paweł Gronowski <[email protected]>
So we can also have darwin binaries in the `dockereng/cli-bin` image.

Signed-off-by: Paweł Gronowski <[email protected]>
@neersighted neersighted merged commit 997cfa6 into docker:master May 21, 2024
88 checks passed
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.

None yet

6 participants