Merge pull request godotengine#98066 from TCROC/fix-android-mono-export

Fix Android mono export with 2 or more cpu architectures fails

Author: Repiteo

modules/mono/editor/GodotTools/GodotTools/Export/ExportPlugin.cs

@@ -216,6 +216,8 @@ private void _ExportBeginImpl(string[] features, bool isDebug, string path, long
 
             bool embedBuildResults = ((bool)GetOption("dotnet/embed_build_outputs") || platform == OS.Platforms.Android) && platform != OS.Platforms.MacOS;
 
+            var exportedJars = new HashSet<string>();
+
             foreach (PublishConfig config in targets)
             {
                 string ridOS = config.RidOS;
@@ -325,14 +327,6 @@ private void _ExportBeginImpl(string[] features, bool isDebug, string path, long
                                     {
                                         string fileName = Path.GetFileName(path);
 
-                                        if (fileName.EndsWith(".jar"))
-                                        {
-                                            // We exclude jar files from the export since they should
-                                            // already be included in the Godot templates, adding them
-                                            // again would cause conflicts.
-                                            return;
-                                        }
-
                                         if (IsSharedObject(fileName))
                                         {
                                             AddSharedObject(path, tags: new string[] { arch },
@@ -343,10 +337,19 @@ private void _ExportBeginImpl(string[] features, bool isDebug, string path, long
                                             return;
                                         }
 
-                                        static bool IsSharedObject(string fileName)
+                                        bool IsSharedObject(string fileName)
                                         {
-                                            if (fileName.EndsWith(".so") || fileName.EndsWith(".a")
-                                             || fileName.EndsWith(".dex"))
+                                            if (fileName.EndsWith(".jar"))
+                                            {
+                                                // Don't export the same jar twice. Otherwise we will have conflicts.
+                                                // This can happen when exporting for multiple architectures. Dotnet
+                                                // stores the jars in .godot/mono/temp/bin/Export[Debug|Release] per
+                                                // target architecture. Jars are cpu agnostic so only 1 is needed.
+                                                var jarName = Path.GetFileName(fileName);
+                                                return exportedJars.Add(jarName);
+                                            }
+
+                                            if (fileName.EndsWith(".so") || fileName.EndsWith(".a") || fileName.EndsWith(".dex"))
                                             {
                                                 return true;
                                             }

…Security.Cryptography.Native.Android.jar …Security.Cryptography.Native.Android.jarplatform/android/java/app/monoLibs/libSystem.Security.Cryptography.Native.Android.jar renamed to modules/mono/thirdparty/libSystem.Security.Cryptography.Native.Android.jar platform/android/java/app/monoLibs/libSystem.Security.Cryptography.Native.Android.jar renamed to modules/mono/thirdparty/libSystem.Security.Cryptography.Native.Android.jar

@@ -70,7 +70,10 @@ dependencies {
     }
 
     // .NET dependencies
-    monoImplementation fileTree(dir: 'monoLibs', include: ['*.jar'])
+    String jar = '../../../../modules/mono/thirdparty/libSystem.Security.Cryptography.Native.Android.jar'
+    if (file(jar).exists()) {
+        monoImplementation files(jar)
+    }
 }
 
 android {