Skip to content
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

Drop MonoPackageManager for APIs older than 9 #2691

Merged
merged 1 commit into from
Feb 4, 2019

Conversation

grendello
Copy link
Contributor

Context: #2690
Fixes: #2690

Xamarin.Android and Mono are no longer capable of running on API levels lower
than 9 (if built with NDK r14) or 16 (if built with NDK r18+). This commit drops
the version of MonoPackageManager targetting API levels < 9.

If an application with its minimum target sdk set to anything < 9 is build it
results in a compilation error when building Java code:

error :  error: method init in class Runtime cannot be applied to given types; [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
error :	Runtime.init ( [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
error :   required: String,String[],String,String[],ClassLoader,String[],String[],String,int,String[] [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
error :   found: String,String[],String,String[],ClassLoader,String[],String[],String [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
error :   reason: actual and formal argument lists differ in length [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]

This is caused by the changes introduced in the runtime performance commit
b90d3ab
which changed the Runtime.init signature but only in the version of
MonoPackageManager targetting the currently supported minimum SDK levels.

Context: dotnet#2690
Fixes: dotnet#2690

Xamarin.Android and Mono are no longer capable of running on API levels lower
than 9 (if built with NDK r14) or 16 (if built with NDK r18+). This commit drops
the version of `MonoPackageManager` targetting API levels < 9.

If an application with its minimum target sdk set to anything < 9 is build it
results in a compilation error when building Java code:

    error :  error: method init in class Runtime cannot be applied to given types; [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
    error :	Runtime.init ( [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
    error :   required: String,String[],String,String[],ClassLoader,String[],String[],String,int,String[] [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
    error :   found: String,String[],String,String[],ClassLoader,String[],String[],String [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
    error :   reason: actual and formal argument lists differ in length [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]

This is caused by the changes introduced in the runtime performance commit
dotnet@b90d3ab
which changed the `Runtime.init` signature but *only* in the version of
`MonoPackageManager` targetting the currently supported minimum SDK levels.
Copy link
Member

@jonathanpeppers jonathanpeppers left a comment

Choose a reason for hiding this comment

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

Looks good I don't see anything else using this file in xamarin-android or monodroid.

@jonpryor jonpryor merged commit 590e53c into dotnet:master Feb 4, 2019
@grendello grendello deleted the drop-packagemanager-api4 branch February 4, 2019 21:02
jonpryor pushed a commit that referenced this pull request Feb 6, 2019
Xamarin.Android and Mono are no longer capable of running on API
levels lower than 9 (if built with NDK r14) or 16 (if built with
NDK r18+).  This commit drops the version of `MonoPackageManager`
targeting API levels < 9.

If an application with its minimum target sdk set to anything < 9 is
built it results in a compilation error when building Java code:

	error :  error: method init in class Runtime cannot be applied to given types; [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
	error :	Runtime.init ( [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
	error :   required: String,String[],String,String[],ClassLoader,String[],String[],String,int,String[] [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
	error :   found: String,String[],String,String[],ClassLoader,String[],String[],String [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]
	error :   reason: actual and formal argument lists differ in length [C:\xms\NPlus1DaysOfMvvmCross\N-04-ValueConverters\Value.Droid\Value.Droid.csproj]

This is caused by the changes introduced in the runtime performance
commit b90d3ab which changed the `Runtime.init()` signature but
*only* in the version of `MonoPackageManager` targeting the currently
supported minimum SDK levels.
@github-actions github-actions bot locked and limited conversation to collaborators Feb 1, 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.

3 participants