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

Bump to mono/opentk/main@daa9b2d5 #7192

Merged
merged 1 commit into from
Jul 26, 2022

Conversation

jonpryor
Copy link
Member

@jonpryor
Copy link
Member Author

On-device tests are failing, e.g. Mono.Android.NET_Tests, Xamarin.Android.RuntimeTests.CustomWidgetTests.UpperAndLowerCaseCustomWidget_FromLibrary_ShouldNotThrowInflateException / Release:

Regression test for widgets with uppercase and lowercase namespace (bug #23880) failed.
Expected: No Exception to be thrown
But was:   (Binary XML file line #1 in Mono.Android.NET_Tests:layout/upper_lower_custom: Binary XML file line #1 in Mono.Android.NET_Tests:layout/upper_lower_custom: Error inflating class mono.android_test.library.CustomTextView)
at Java.Interop.JniEnvironment.InstanceMethods.CallObjectMethod(JniObjectReference , JniMethodInfo , JniArgumentValue* )
at Java.Interop.JniPeerMembers.JniInstanceMethods.InvokeVirtualObjectMethod(String , IJavaPeerable , JniArgumentValue* )
at Android.Views.LayoutInflater.Inflate(Int32 , ViewGroup )
at Xamarin.Android.RuntimeTests.CustomWidgetTests.<>c.<UpperAndLowerCaseCustomWidget_FromLibrary_ShouldNotThrowInflateException>b__2_0()
at NUnit.Framework.Constraints.VoidInvocationDescriptor.Invoke()
at NUnit.Framework.Constraints.ExceptionInterceptor.Intercept(Object )
--- End of managed Java.Lang.RuntimeException stack trace ---
android.view.InflateException: Binary XML file line #1 in Mono.Android.NET_Tests:layout/upper_lower_custom: Binary XML file line #1 in Mono.Android.NET_Tests:layout/upper_lower_custom: Error inflating class mono.android_test.library.CustomTextView
Caused by: android.view.InflateException: Binary XML file line #1 in Mono.Android.NET_Tests:layout/upper_lower_custom: Error inflating class mono.android_test.library.CustomTextView
Caused by: java.lang.ClassNotFoundException: mono.android_test.library.CustomTextView
at java.lang.Class.classForName(Native Method)
at java.lang.Class.forName(Class.java:454)
at android.view.LayoutInflater.createView(LayoutInflater.java:815)
at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:1006)
at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:961)
at android.view.LayoutInflater.rInflate(LayoutInflater.java:1123)
at android.view.LayoutInflater.rInflateChildren(LayoutInflater.java:1084)
at android.view.LayoutInflater.inflate(LayoutInflater.java:682)
at android.view.LayoutInflater.inflate(LayoutInflater.java:534)
at android.view.LayoutInflater.inflate(LayoutInflater.java:481)
at crc643df67da7b13bb6b1.TestInstrumentation_1.n_onStart(Native Method)
at crc643df67da7b13bb6b1.TestInstrumentation_1.onStart(TestInstrumentation_1.java:35)
at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:2189)
Caused by: java.lang.ClassNotFoundException: mono.android_test.library.CustomTextView
... 13 more

This makes no sense to me; mono.android_test.library.CustomTextView is presumably coming from CustomTextView.cs, but the type should be using a crc… package name, not mono.android_test.library!

Additionally, this works on main (yay; 938b2cb is fully green!), and the only thing changing here is an opentk bump, and I don't see how opentk would have anything to do here.

Very confusing.

I'm just gonna start a new build.

@jonpryor
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Changes: mono/opentk@50128f7...daa9b2d

  * mono/opentk@daa9b2d5: [nuget] use $(PackageLicenseExpression), not $(PackageLicenseUrl) (dotnet#35)
  * mono/opentk@351eb656: [ci] Use new EO compliant build pools (dotnet#33)
  * mono/opentk@95ab20b1: Try to fix DllNotFoundException under .net 6 (dotnet#32)
  * mono/opentk@3de39a40: Merge pull request dotnet#30 from dellis1972/proguard
  * mono/opentk@79ea8095: rework
  * mono/opentk@3311a8ac: Add Proguard config support
@jonpryor jonpryor merged commit 2473218 into dotnet:main Jul 26, 2022
grendello added a commit to grendello/xamarin-android that referenced this pull request Aug 1, 2022
* main:
  LEGO: Merge pull request 7221
  LEGO: Merge pull request 7219
  [Xamarin.Android.Build.Tasks] use `ToJniName(type, cache)` (dotnet#7211)
  [docs] Synchronize with MicrosoftDocs/xamarin-docs (dotnet#7208)
  [Mono.Android] Remove System.Linq usage (dotnet#7210)
  Bump to Android NDK r25 (dotnet#6764)
  Bump to mono/opentk@daa9b2d5 (dotnet#7192)
  [Mono.Android] Optional NTLMv2 support in AndroidMessageHandler (dotnet#6999)
  Bump to dotnet/installer@53587f9 7.0.100-rc.1.22374.1 (dotnet#7198)
grendello added a commit to grendello/xamarin-android that referenced this pull request Aug 1, 2022
* mm-codegen:
  LEGO: Merge pull request 7221
  LEGO: Merge pull request 7219
  [Xamarin.Android.Build.Tasks] use `ToJniName(type, cache)` (dotnet#7211)
  [docs] Synchronize with MicrosoftDocs/xamarin-docs (dotnet#7208)
  [Mono.Android] Remove System.Linq usage (dotnet#7210)
  Bump to Android NDK r25 (dotnet#6764)
  Bump to mono/opentk@daa9b2d5 (dotnet#7192)
  [Mono.Android] Optional NTLMv2 support in AndroidMessageHandler (dotnet#6999)
  Bump to dotnet/installer@53587f9 7.0.100-rc.1.22374.1 (dotnet#7198)
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 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