Add docker image w/ both Android SDK and an x64 crossroot #901
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Sigh. How do I get the intra-dependency to show up? |
jkoritzinsky
approved these changes
Jul 3, 2023
|
You need to depend on the -local versions of the base images and add local tags for the base images in the manifest if they aren't there already. |
|
|
This was referenced Jul 5, 2023
|
This seems OK now? (thanks @lbussell) Can I get a re-review @jkoritzinsky? |
jkoritzinsky
approved these changes
Jul 8, 2023
directhex
added a commit
to directhex/dotnet-buildtools-prereqs-docker
that referenced
this pull request
Aug 18, 2023
This is required in some cases - e.g. compiling node.js for ARM64/Alpine, and the ever-broken ARM64 runtime perf lane (where we need headers for ARM64 available alongside linkable libraries for AMD64, depending on the build phase). As is, we've achieved the same result largely by accident in the pre-Mariner build images, where we have enough x64 in the host OS to link successfully, and the ARM64 files in the crossroot. The precedent on this is dotnet#901 which fixed our Android cross-compiler builds, by offering both Android and $HOST in the same system If at some point in the future we move from building on x64 to building on ARM64, we will need the equivalent images for that.
directhex
added a commit
that referenced
this pull request
Aug 21, 2023
This is required in some cases - e.g. compiling node.js for ARM64/Alpine, and the ever-broken ARM64 runtime perf lane (where we need headers for ARM64 available alongside linkable libraries for AMD64, depending on the build phase). As is, we've achieved the same result largely by accident in the pre-Mariner build images, where we have enough x64 in the host OS to link successfully, and the ARM64 files in the crossroot. The precedent on this is #901 which fixed our Android cross-compiler builds, by offering both Android and $HOST in the same system If at some point in the future we move from building on x64 to building on ARM64, we will need the equivalent images for that. Co-authored-by: Matt Thalman <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
(for AllSubsets jobs)