Skip to content

Conversation

@dellis1972
Copy link
Contributor

No description provided.

@jonpryor
Copy link
Contributor

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 jonpryor added the do-not-merge Do not merge this PR label Feb 16, 2023
@tondat
Copy link
Contributor

tondat commented Feb 16, 2023

@jonpryor working on that.

The idea is update the feed for 17.6 ?

@tondat
Copy link
Contributor

tondat commented Feb 16, 2023

@jonpryor this is the proposed feed with the changes: https://github.com/xamarin/android-sdk-installer/pull/765
if all is ok we can update the feed.

@tondat
Copy link
Contributor

tondat commented Feb 28, 2023

@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.

@jonpryor
Copy link
Contributor

@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:

  1. Why is platform-tools 34.0.0 already stable? (More rhetorical than anything, but this is really weirding me out.)
  2. Do we work with platform-tools 34.0.0? As far as I know, we have not attempted to to go through the xamarin-android unit tests with platform-tools 34.0.0, and "normally" I would expect this to happen as part of API-34 binding, which is still in-progress. @jpobst? :-D

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.

@jonpryor
Copy link
Contributor

@jpobst points out that I'm overthinking this: new platform-tools versions regularly come out in February; see the SDK Platform Tools release notes:

  • 34.0.0 (February 2023)
  • 33.0.0 (February 2022)
  • 32.0.0 (January 2022)
  • 31.0.0 (February 2021)

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.

@jonpryor
Copy link
Contributor

jonpryor commented Mar 9, 2023

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 $(AndroidSdkPlatformToolsVersion) to all the relevant d17-5 branches, including (but not limited to?):

  • xamarin-android-tools/d17-5
  • xamarin-android/d17-5
  • monodroid/d17-5
  • androidtools/d17-5
  • android-sdk-installer/d17-5 (???)

@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.)

@jonpryor
Copy link
Contributor

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.
jonpryor added a commit to jonpryor/xamarin-android that referenced this pull request Apr 10, 2023
@jonpryor jonpryor removed the do-not-merge Do not merge this PR label Apr 19, 2023
@jonpryor jonpryor merged commit 8bc0750 into dotnet:main Apr 19, 2023
jonpryor pushed a commit that referenced this pull request Apr 19, 2023
)

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.
jonpryor added a commit to jonpryor/xamarin-android that referenced this pull request Apr 19, 2023
jonpryor added a commit to jonpryor/xamarin-android that referenced this pull request Apr 19, 2023
jonpryor added a commit to jonpryor/xamarin-android that referenced this pull request Apr 20, 2023
jonpryor added a commit to dotnet/android that referenced this pull request Apr 20, 2023
jonpryor added a commit to dotnet/android that referenced this pull request Apr 21, 2023
jonpryor added a commit to dotnet/android that referenced this pull request Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants