-
Notifications
You must be signed in to change notification settings - Fork 43
Package and bundle Android SDKs into the windows toolchain installer #297
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
Changes from 10 commits
ec6c2ea
e242ac3
4880823
3c0215e
4ec8306
b688c30
d76bdea
7f70a53
e861f2d
301bcc1
c384797
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
<Project Sdk="WixToolset.Sdk/4.0.1"> | ||
<PropertyGroup> | ||
<OutputName>android_sdk.$(ProductArchitecture)</OutputName> | ||
<Platform>x86</Platform> | ||
|
||
<SwiftShimsPath>$(SDK_ROOT)\usr\lib\swift\shims</SwiftShimsPath> | ||
|
||
<DefineConstants> | ||
$(DefineConstants); | ||
SwiftShimsPath=$(SwiftShimsPath); | ||
</DefineConstants> | ||
</PropertyGroup> | ||
|
||
<ItemGroup> | ||
<PackageReference Include="WixToolset.Heat" Version="4.0.1" /> | ||
</ItemGroup> | ||
|
||
<ItemGroup> | ||
<HarvestDirectory Include="$(SwiftShimsPath)"> | ||
<ComponentGroupName>SwiftShims</ComponentGroupName> | ||
<DirectoryRefId>AndroidSDK_usr_lib_swift_shims</DirectoryRefId> | ||
<PreprocessorVariable>var.SwiftShimsPath</PreprocessorVariable> | ||
<SuppressCom>true</SuppressCom> | ||
<SuppressRegistry>true</SuppressRegistry> | ||
<SuppressRootDirectory>true</SuppressRootDirectory> | ||
</HarvestDirectory> | ||
</ItemGroup> | ||
</Project> |
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -38,6 +38,10 @@ | |
<Variable Name="OptionsInstallRedistAMD64" bal:Overridable="yes" Persisted="yes" Value="1" /> | ||
<Variable Name="OptionsInstallSdkArm64" bal:Overridable="yes" Persisted="yes" Value="1" /> | ||
<Variable Name="OptionsInstallRedistArm64" bal:Overridable="yes" Persisted="yes" Value="1" /> | ||
<Variable Name="OptionsInstallAndroidSdkArm64" bal:Overridable="yes" Persisted="yes" Value="1" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is the default value specified here? I think the Android SDK should be opt-in. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let's make arm64 and x86_64 on by default, but not the others ? |
||
<Variable Name="OptionsInstallAndroidSdkAMD64" bal:Overridable="yes" Persisted="yes" Value="1" /> | ||
<Variable Name="OptionsInstallAndroidSdkArm" bal:Overridable="yes" Persisted="yes" Value="0" /> | ||
<Variable Name="OptionsInstallAndroidSdkX86" bal:Overridable="yes" Persisted="yes" Value="0" /> | ||
|
||
<!-- | ||
For the online bundle, we need to provide a download URL for each package and its .cabs. | ||
|
@@ -119,6 +123,42 @@ | |
<MsiProperty Name="INSTALLREDIST" Value="[OptionsInstallRedistArm64]" /> | ||
</MsiPackage> | ||
<?endif?> | ||
|
||
<?if $(INCLUDE_ARM64_ANDROID_SDK) == true ?> | ||
<MsiPackage | ||
SourceFile="!(bindpath.android_sdk_aarch64)\android_sdk.aarch64.msi" | ||
InstallCondition="OptionsInstallAndroidSdkArm64" | ||
DownloadUrl="$(BaseReleaseDownloadUrl)/{2}"> | ||
<MsiProperty Name="INSTALLROOT" Value="[InstallRoot]" /> | ||
</MsiPackage> | ||
<?endif?> | ||
|
||
<?if $(INCLUDE_x86_64_ANDROID_SDK) == true ?> | ||
<MsiPackage | ||
SourceFile="!(bindpath.android_sdk_x86_64)\android_sdk.x86_64.msi" | ||
InstallCondition="OptionsInstallAndroidSdkAMD64" | ||
DownloadUrl="$(BaseReleaseDownloadUrl)/{2}"> | ||
<MsiProperty Name="INSTALLROOT" Value="[InstallRoot]" /> | ||
</MsiPackage> | ||
<?endif?> | ||
|
||
<?if $(INCLUDE_ARM_ANDROID_SDK) == true ?> | ||
<MsiPackage | ||
SourceFile="!(bindpath.android_sdk_armv7)\android_sdk.armv7.msi" | ||
InstallCondition="OptionsInstallAndroidSdkArm" | ||
DownloadUrl="$(BaseReleaseDownloadUrl)/{2}"> | ||
<MsiProperty Name="INSTALLROOT" Value="[InstallRoot]" /> | ||
</MsiPackage> | ||
<?endif?> | ||
|
||
<?if $(INCLUDE_X86_ANDROID_SDK) == true ?> | ||
<MsiPackage | ||
SourceFile="!(bindpath.android_sdk_i686)\android_sdk.i686.msi" | ||
InstallCondition="OptionsInstallAndroidSdkX86" | ||
DownloadUrl="$(BaseReleaseDownloadUrl)/{2}"> | ||
<MsiProperty Name="INSTALLROOT" Value="[InstallRoot]" /> | ||
</MsiPackage> | ||
<?endif?> | ||
</Chain> | ||
</Bundle> | ||
</Wix> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nicer to swap this around to
ANDROID_...
IMO