-
Couldn't load subscription status.
- Fork 31
Bump 'AndroidSdkPlatformToolsVersion' to 34.0.0 #203
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
Conversation
|
We cannot bump this default change until android-sdk-installer likewise supports platform-tools 34.0.0. @tondat: who's backlog can we put this on? :-) |
|
@jonpryor working on that. The idea is update the feed for 17.6 ? |
|
@jonpryor this is the proposed feed with the changes: https://github.com/xamarin/android-sdk-installer/pull/765 |
|
@jonpryor these PR has the change on willow: https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/454760 and this PR has the proposed new manifest https://github.com/xamarin/android-sdk-installer/pull/765 waiting for your approval to proceed. |
|
@tondat: so here's the philosophical conundrum: API-34 is still in preview, and won't be stable until July or later. However, that's the platform package; here, we're talking about the platform-tools package, and here we have a bunch of unknowns:
Given (2), we should not update Willow to install only platform-tools 34.0.0 now, and possibly not until API-34 is the default (2023- November?). xamarin/android-sdk-installer#765 should be "fine" (-ish; I have some comments to leave there), as it is only additive, not replacing. Which still leaves this PR: we shouldn't merge it until we're able to make platform-tools 34.0.0 the default everywhere. Maybe that's "sooner" -- once we've done a complete test pass using platform-tools 34.0.0 and have verified it works with API-31…API-33 apps -- or maybe that's November, but it's certainly not now. |
|
@jpobst points out that I'm overthinking this: new platform-tools versions regularly come out in February; see the SDK Platform Tools release notes:
I am thus freaking out more than is strictly necessary, and this bump is likely safe. That said, we should still try a bump in xamarin-android first, just to verify that nothing breaks. Nothing should break, but there is no harm to being careful here. |
|
On the "Does xamarin-android Build™ with platform-tools 34.0.0" front, the answer is a tentative "Yes":
If both of those PRs are fully green, then Yes, we work with platform-tools 34.0.0. That said… if the intention is also to change the VS installer, then we will also need to apply the changes to
@tondat: is that a set of changes you are comfortable with? Or do we need to consider "bumping" monodroid/d17-5 to use "newer" androidtools & android-sdk-installer branches? (Background: xamarin-android/main is no longer completely building Classic Xamarin.Android; xamarin-android/d17-5 is our ongoing maintenance branch for Classic Xamarin.Android. We could plausibly create a set of d17-6 branches from d17-5 -- for xamarin-android-tools, xamarin-android, monodroid -- but this is something we need to discuss.) |
|
Platform-tools 34.0.1 is current latest version on https://dl-ssl.google.com/android/repository/repository2-3.xml |
34.0.1 is currently the only version listed in repository2-3.xml.
) Context: https://dl-ssl.google.com/android/repository/repository2-3.xml Context: https://developer.android.com/studio/releases/platform-tools As of 2023-Mar-03 (according to comments in `repository2-3.xml`), the Android SDK platform-tools package version 34.0.1 is now (1) stable, and (2) the *only* platform-tools package listed in `repository2-3.xml`. Changes from 33.0.2 include: * Various changes to `adb` * Various changes to `fastboot` Update `$(AndroidSdkPlatformToolsVersion)` to 34.0.1 so that the Android SDK platform-tools 34.0.1 package is the default version used in Xamarin.Android / .NET Android builds. TODO: Visual Studio 17.7 will be updated to install platform-tools 34.0.1 by default as well.
Changes: http://github.com/xamarin/xamarin-android-tools/compare/dbe42bf1688e79e1ca474dff4a1f8060158cafd9...8bc075035818e7731df79ec688e2c6b154a29b4f * dotnet/android-tools@8bc0750: [Xamarin.Android.Tools.AndroidSdk] Update platform-tools for API-T (dotnet/android-tools#203)
Changes: dotnet/android-tools@9f02d77...ca1552d * dotnet/android-tools@ca1552d: [Xamarin.Android.Tools.AndroidSdk] Update platform-tools for API-T (dotnet/android-tools#203)
Changes: http://github.com/xamarin/xamarin-android-tools/compare/dbe42bf1688e79e1ca474dff4a1f8060158cafd9...8bc075035818e7731df79ec688e2c6b154a29b4f * dotnet/android-tools@8bc0750: [Xamarin.Android.Tools.AndroidSdk] Update platform-tools for API-T (dotnet/android-tools#203)
Changes: dotnet/android-tools@9f02d77...ca1552d * dotnet/android-tools@ca1552d: [Xamarin.Android.Tools.AndroidSdk] Update platform-tools for API-T (dotnet/android-tools#203)
Changes: http://github.com/xamarin/xamarin-android-tools/compare/dbe42bf1688e79e1ca474dff4a1f8060158cafd9...8bc075035818e7731df79ec688e2c6b154a29b4f * dotnet/android-tools@8bc0750: [Xamarin.Android.Tools.AndroidSdk] Update platform-tools for API-T (dotnet/android-tools#203)
Changes: dotnet/android-tools@9f02d77...ca1552d * dotnet/android-tools@ca1552d: [Xamarin.Android.Tools.AndroidSdk] Update platform-tools for API-T (dotnet/android-tools#203)
Changes: dotnet/android-tools@9f02d77...ca1552d * dotnet/android-tools@ca1552d: [Xamarin.Android.Tools.AndroidSdk] Update platform-tools for API-T (dotnet/android-tools#203)
No description provided.