-
Notifications
You must be signed in to change notification settings - Fork 526
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
[build] Remove MONOANDROID, NET, and NETCOREAPP directives #8434
Conversation
0fff586
to
c24c3c2
Compare
...n.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/Tasks/CheckClientHandlerTypeTests.cs
Outdated
Show resolved
Hide resolved
Removes the conditional compiler directives used to swap between `.NET` and `MonoAndroid`. These are no longer needed in main as `.NET` is the only supported target.
c24c3c2
to
a3c5fbe
Compare
@@ -91,7 +89,7 @@ public bool FitsSystemWindows () | |||
} | |||
#endif | |||
|
|||
#if NET && ANDROID_34 | |||
#if ANDROID_34 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think ANDROID_34
could be deleted as well. We won't be making a net8.0-android33
or net9.0-android33
, etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we will want to go through and clean this up in a future PR? Given that main still allows you to build Mono.Android.csproj for ~any supported framework version I think it makes sense to leave these in for now.
@@ -4,7 +4,7 @@ namespace Android.Views | |||
{ | |||
partial class WindowManagerLayoutParams | |||
{ | |||
#if NET && ANDROID_34 | |||
#if ANDROID_34 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ANDROID_34
Removes the conditional compiler directives used to swap between
.NET
and
MonoAndroid
. These are no longer needed in main as.NET
is theonly supported target.