From ef4bf56787fd146bb3a39abd2650cf8df0cccfdb Mon Sep 17 00:00:00 2001 From: Jonathan Peppers Date: Tue, 31 Oct 2017 09:06:45 -0500 Subject: [PATCH] [build] update SDK tools URLs On Windows, the `lint` tool installed by the build was broken. This causes several tests to fail on Windows verifying features around `lint` After looking into it, it appears the URLs used to download the SDK tools were not found in: https://dl.google.com/android/repository/repository-12.xml The correct URLs appear to be of the form `tools_rx.x.x-platform.zip`. Note Google uses `macosx` in this file name instead of `darwin`. Not sure what version of the SDK tools were being used previously, they must be a prerelease or nightly release. Other changes: - The command `sdkmanager --license` is now `sdkmanager --update` --- build-tools/android-toolchain/android-toolchain.projitems | 6 +++--- .../AcceptAndroidSdkLicenses.cs | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/build-tools/android-toolchain/android-toolchain.projitems b/build-tools/android-toolchain/android-toolchain.projitems index 32ab3952e9e..e4467063f6d 100644 --- a/build-tools/android-toolchain/android-toolchain.projitems +++ b/build-tools/android-toolchain/android-toolchain.projitems @@ -19,7 +19,7 @@ Linux platform-tools - + Linux tools @@ -38,7 +38,7 @@ Darwin platform-tools - + Darwin tools @@ -57,7 +57,7 @@ Windows platform-tools - + Windows tools diff --git a/build-tools/xa-prep-tasks/Xamarin.Android.BuildTools.PrepTasks/AcceptAndroidSdkLicenses.cs b/build-tools/xa-prep-tasks/Xamarin.Android.BuildTools.PrepTasks/AcceptAndroidSdkLicenses.cs index 467fe24310e..469aaa548aa 100644 --- a/build-tools/xa-prep-tasks/Xamarin.Android.BuildTools.PrepTasks/AcceptAndroidSdkLicenses.cs +++ b/build-tools/xa-prep-tasks/Xamarin.Android.BuildTools.PrepTasks/AcceptAndroidSdkLicenses.cs @@ -28,7 +28,7 @@ public override bool Execute () string _; var path = Which.GetProgramLocation ("sdkmanager", out _, new [] { Path.Combine (AndroidSdkDirectory, "tools", "bin") }); - var psi = new ProcessStartInfo (path, "--licenses") { UseShellExecute = false, RedirectStandardInput = true }; + var psi = new ProcessStartInfo (path, "--update") { UseShellExecute = false, RedirectStandardInput = true }; var proc = Process.Start (psi); for (int i = 0; i < 10; i++) proc.StandardInput.WriteLine ('y');