Skip to content

Conversation

@jpobst
Copy link
Contributor

@jpobst jpobst commented May 12, 2023

Remove monoandroid10 target framework from Mono.Android.csproj and Mono.Android.Export.csproj.

This enables removing net472 from Java.Interop: dotnet/java-interop#1099.

@jpobst jpobst marked this pull request as ready for review May 12, 2023 18:38
@jpobst jpobst requested review from jonpryor and pjcollins and removed request for dellis1972, jonathanpeppers and jonpryor May 12, 2023 18:38

<PropertyGroup>
<TargetFrameworks>monoandroid10;$(DotNetTargetFramework)</TargetFrameworks>
<TargetFrameworks>$(DotNetTargetFramework)</TargetFrameworks>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we may be able to skip some extra outer/inner build processing by changing this to TargetFramework now, though I'm not sure if it makes much of a difference.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this suggestion, and it probably applies to a lot of projects. I'll include this in some future cleanups. 👍

@jonpryor jonpryor merged commit af57c52 into main May 15, 2023
@jonpryor jonpryor deleted the monoandroid10 branch May 15, 2023 19:04
grendello added a commit to grendello/xamarin-android that referenced this pull request May 16, 2023
* main:
  Bump to xamarin/xamarin-android-tools/main@3b8c467 (dotnet#8039)
  [Xamarin.Android.Build.Tasks] skip `proguard.txt` files in !app (dotnet#8025)
  [Mono.Android] Remove `monoandroid10` target framework (dotnet#8035)
@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 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.

6 participants