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

Add build configuration to generate runtime packs for Android #34192

Merged

Conversation

mdh1418
Copy link
Member

@mdh1418 mdh1418 commented Mar 27, 2020

This PR adds support for generating Android runtime packs, which is intended to be consumed by the Xamarin Android SDK. It takes after #34050 and when that is merged, this PR will rebase off it.

eng/Subsets.props Outdated Show resolved Hide resolved
eng/Subsets.props Outdated Show resolved Hide resolved
@@ -187,6 +189,13 @@
<TargetsUnix>true</TargetsUnix>
</PropertyGroup>
</When>
<When Condition="$(OutputRid.StartsWith('android'))">
Copy link
Member

@steveisok steveisok Mar 27, 2020

Choose a reason for hiding this comment

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

@akoeplinger is <TargetsLinux> required here?

Copy link
Member

Choose a reason for hiding this comment

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

probably not required if it's not used but in general the Targets* seem to follow the RID graph which has Android->Linux->Unix dependency.

eng/liveBuilds.targets Outdated Show resolved Hide resolved
@mdh1418 mdh1418 force-pushed the mdhwang/gen_runtime_packs_for_android branch 3 times, most recently from 73baade to c8b2152 Compare April 2, 2020 18:15
@steveisok steveisok changed the title [WIP] Add build configuration to generate runtime packs for Android Add build configuration to generate runtime packs for Android Apr 2, 2020
@steveisok steveisok requested a review from a team April 2, 2020 22:03
@mdh1418 mdh1418 force-pushed the mdhwang/gen_runtime_packs_for_android branch from cee1712 to 48ae937 Compare April 3, 2020 18:27
@steveisok
Copy link
Member

@steveisok steveisok requested review from dagood and safern April 4, 2020 00:11
@mdh1418 mdh1418 force-pushed the mdhwang/gen_runtime_packs_for_android branch from 48ae937 to 84923a0 Compare April 6, 2020 14:46
@mdh1418 mdh1418 force-pushed the mdhwang/gen_runtime_packs_for_android branch from 84923a0 to 7ec7c75 Compare April 6, 2020 14:49
@mdh1418 mdh1418 force-pushed the mdhwang/gen_runtime_packs_for_android branch from 7ec7c75 to 0ffa5d3 Compare April 6, 2020 14:54
@mdh1418 mdh1418 force-pushed the mdhwang/gen_runtime_packs_for_android branch from 0ffa5d3 to 5ec3cc3 Compare April 6, 2020 14:56
@mdh1418 mdh1418 merged commit 99aa064 into dotnet:master Apr 6, 2020
@mdh1418 mdh1418 deleted the mdhwang/gen_runtime_packs_for_android branch April 6, 2020 16:30
@ghost ghost locked as resolved and limited conversation to collaborators Dec 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants