Skip to content

Conversation

@jonathanpeppers
Copy link
Member

@jonathanpeppers jonathanpeppers commented Jun 3, 2021

Changes: https://github.com/xamarin/monodroid/compare/76c04cd1...1f2ce156

  • [tools/msbuild] Missing translations for XA0135
  • Bump to 032d840, xamarin/androidtools@355d015
  • [build] fix dotnet tool install command
  • [tools/fastdev] Rework Unix timestamp calculation code in xamarin.find
  • [tools/fastdev] Add error checking when writing data to disk.
  • [tools/msbuild] selects backup RIDs for .NET 6
  • Bump android-sdk-installer to use Mono.Unix
  • [tools/msbuild] only run _GetPrimaryCpuAbi for Fast Dev

Other changes:

  • InstantRunNativeLibrary will need to set the ABI in the .csproj
    in order for the test to pass on .NET 6. It's use of
    BaseTest.DeviceAbi would not match for an x86_64 emulator.

@jonathanpeppers jonathanpeppers requested a review from jonpryor as a code owner June 3, 2021 21:18
@jonathanpeppers
Copy link
Member Author

Maybe don't merge this one yet, I found an issue testing this:

NotifySync SkipCopyFile obj\Debug\net6.0-android\android\assets\HelloAndroid.dll=>HelloAndroid.dll file is up to date. [0ms]
NotifySync CopyFile obj\Debug\net6.0-android\android\assets\HelloAndroid.pdb. [17ms]

I added a line like this to my MainActivity.OnCreate():

System.Console.WriteLine("WOOT!");

But I didn't get the change.

@jonathanpeppers jonathanpeppers marked this pull request as draft June 3, 2021 22:05
@jonathanpeppers jonathanpeppers changed the title Bump to xamarin/monodroid/main@65b7b2dd Bump to xamarin/monodroid/main@48843fcb Jun 7, 2021
@jonathanpeppers jonathanpeppers marked this pull request as ready for review June 7, 2021 17:54
@jonathanpeppers
Copy link
Member Author

The issue in #5991 (comment) should be working now.

I believe I was only seeing the problem in an earlier incarnation of https://github.com/xamarin/monodroid/pull/1205.

@jonathanpeppers
Copy link
Member Author

Latest issue here, is we should only be doing this block when Fast Deployment is enabled...

https://github.com/xamarin/monodroid/blob/48843fcb2c224fb2d5d2e65299c25abeb592445a/tools/msbuild/Xamarin.Android.Common.Debugging.targets#L408-L413

@jonathanpeppers jonathanpeppers marked this pull request as draft June 8, 2021 14:57
Changes: xamarin/monodroid@76c04cd...1f2ce15

* [tools/msbuild] Missing translations for XA0135
* Bump to dotnet/android@032d840, xamarin/androidtools@355d015
* [build] fix dotnet tool install command
* [tools/fastdev] Rework Unix timestamp calculation code in xamarin.find
* [tools/fastdev] Add error checking when writing data to disk.
* [tools/msbuild] <GetPrimaryCpuAbi/> selects backup RIDs for .NET 6
* Bump android-sdk-installer to use Mono.Unix
* [tools/msbuild] only run _GetPrimaryCpuAbi for Fast Dev

Other changes:

* `InstantRunNativeLibrary` will need to set the ABI in the `.csproj`
  in order for the test to pass on .NET 6. It's use of
  `BaseTest.DeviceAbi` would not match for an x86_64 emulator.
@jonathanpeppers jonathanpeppers changed the title Bump to xamarin/monodroid/main@48843fcb Bump to xamarin/monodroid/main@1f2ce156 Jun 8, 2021
@jonathanpeppers
Copy link
Member Author

Closing in favor of: #5971

@github-actions github-actions bot locked and limited conversation to collaborators Jan 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant