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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

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 pawel.gronowski@docker.com

@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 <pawel.gronowski@docker.com>
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
So we can also have darwin binaries in the `dockereng/cli-bin` image.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
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

5 participants