[android] Fix build id (#3066)

Reviewed-on: https://git.eden-emu.dev/eden-emu/eden/pulls/3066
Reviewed-by: crueter <crueter@eden-emu.dev>
Reviewed-by: CamilleLaVey <camillelavey99@gmail.com>
Co-authored-by: Caio Oliveira <caiooliveirafarias0@gmail.com>
Co-committed-by: Caio Oliveira <caiooliveirafarias0@gmail.com>

src/android/app/build.gradle.kts

@@ -61,7 +61,6 @@ android {
         minSdk = 24
         targetSdk = 36
         versionName = getGitVersion()
-
         versionCode = autoVersion
 
         ndk {
@@ -69,9 +68,6 @@ android {
             abiFilters += listOf("arm64-v8a")
         }
 
-        buildConfigField("String", "GIT_HASH", "\"${getGitHash()}\"")
-        buildConfigField("String", "BRANCH", "\"${getBranch()}\"")
-
         externalNativeBuild {
             cmake {
                 val extraCMakeArgs =
@@ -92,14 +88,14 @@ android {
                         "-DYUZU_TESTS=OFF",
                         "-DDYNARMIC_TESTS=OFF",
                         *extraCMakeArgs.toTypedArray()
-                ))
+                    )
+                )
 
                 abiFilters("arm64-v8a")
             }
         }
     }
 
-
     val keystoreFile = System.getenv("ANDROID_KEYSTORE_FILE")
     signingConfigs {
         if (keystoreFile != null) {
@@ -162,32 +158,6 @@ android {
         }
     }
 
-        // this is really annoying but idk any other ways to fix this behavior
-    applicationVariants.all {
-        val variant = this
-        when {
-            variant.flavorName == "legacy" && variant.buildType.name == "debug" -> {
-                variant.resValue("string", "app_name_suffixed", "Eden Legacy Debug")
-            }
-            variant.flavorName == "mainline" && variant.buildType.name == "debug" -> {
-                variant.resValue("string", "app_name_suffixed", "Eden Debug")
-            }
-            variant.flavorName == "genshinSpoof" && variant.buildType.name == "debug" -> {
-                variant.resValue("string", "app_name_suffixed", "Eden Optimized Debug")
-            }
-            variant.flavorName == "legacy" && variant.buildType.name == "relWithDebInfo" -> {
-                variant.resValue("string", "app_name_suffixed", "Eden Legacy Debug Release")
-            }
-            variant.flavorName == "mainline" && variant.buildType.name == "relWithDebInfo" -> {
-                variant.resValue("string", "app_name_suffixed", "Eden Debug Release")
-            }
-            variant.flavorName == "genshinSpoof" && variant.buildType.name == "relWithDebInfo" -> {
-                variant.resValue("string", "app_name_suffixed", "Eden Optimized Debug Release")
-            }
-        }
-    }
-
-    android {
     flavorDimensions.add("version")
     productFlavors {
         create("mainline") {
@@ -219,6 +189,30 @@ android {
             }
         }
     }
+
+    // this is really annoying but idk any other ways to fix this behavior
+    applicationVariants.all {
+        val variant = this
+        when {
+            variant.flavorName == "legacy" && variant.buildType.name == "debug" -> {
+                variant.resValue("string", "app_name_suffixed", "Eden Legacy Debug")
+            }
+            variant.flavorName == "mainline" && variant.buildType.name == "debug" -> {
+                variant.resValue("string", "app_name_suffixed", "Eden Debug")
+            }
+            variant.flavorName == "genshinSpoof" && variant.buildType.name == "debug" -> {
+                variant.resValue("string", "app_name_suffixed", "Eden Optimized Debug")
+            }
+            variant.flavorName == "legacy" && variant.buildType.name == "relWithDebInfo" -> {
+                variant.resValue("string", "app_name_suffixed", "Eden Legacy Debug Release")
+            }
+            variant.flavorName == "mainline" && variant.buildType.name == "relWithDebInfo" -> {
+                variant.resValue("string", "app_name_suffixed", "Eden Debug Release")
+            }
+            variant.flavorName == "genshinSpoof" && variant.buildType.name == "relWithDebInfo" -> {
+                variant.resValue("string", "app_name_suffixed", "Eden Optimized Debug Release")
+            }
+        }
     }
 
     externalNativeBuild {
@@ -284,7 +278,6 @@ dependencies {
     implementation("androidx.core:core-splashscreen:1.0.1")
     implementation("com.fasterxml.jackson.module:jackson-module-kotlin:2.17.2")
     implementation("androidx.window:window:1.3.0")
-    implementation("androidx.constraintlayout:constraintlayout:2.2.1")
     implementation("androidx.swiperefreshlayout:swiperefreshlayout:1.1.0")
     implementation("org.commonmark:commonmark:0.22.0")
     implementation("androidx.navigation:navigation-fragment-ktx:2.8.9")
@@ -302,7 +295,9 @@ fun runGitCommand(command: List<String>): String {
             .directory(project.rootDir)
             .redirectOutput(ProcessBuilder.Redirect.PIPE)
             .redirectError(ProcessBuilder.Redirect.PIPE)
-            .start().inputStream.bufferedReader().use { it.readText() }
+            .start()
+            .inputStream.bufferedReader()
+            .use { it.readText() }
             .trim()
     } catch (e: Exception) {
         logger.error("Cannot find git")
@@ -326,9 +321,3 @@ fun getGitVersion(): String {
     }
     return versionName.ifEmpty { "0.0" }
 }
-
-fun getGitHash(): String =
-    runGitCommand(listOf("git", "rev-parse", "--short", "HEAD")).ifEmpty { "dummy-hash" }
-
-fun getBranch(): String =
-    runGitCommand(listOf("git", "rev-parse", "--abbrev-ref", "HEAD")).ifEmpty { "dummy-hash" }

src/android/app/src/main/java/org/yuzu/yuzu_emu/NativeLibrary.kt

@@ -227,6 +227,11 @@ object NativeLibrary {
      */
     external fun isUpdateCheckerEnabled(): Boolean
 
+    /**
+     * Returns the build version generated by CMake (BUILD_VERSION).
+     */
+    external fun getBuildVersion(): String
+
     enum class CoreError {
         ErrorSystemFiles,
         ErrorSavestate,

src/android/app/src/main/java/org/yuzu/yuzu_emu/fragments/AboutFragment.kt

@@ -29,6 +29,7 @@ import org.yuzu.yuzu_emu.R
 import org.yuzu.yuzu_emu.databinding.FragmentAboutBinding
 import org.yuzu.yuzu_emu.model.HomeViewModel
 import org.yuzu.yuzu_emu.utils.ViewUtils.updateMargins
+import org.yuzu.yuzu_emu.NativeLibrary
 
 class AboutFragment : Fragment() {
     private var _binding: FragmentAboutBinding? = null
@@ -78,11 +79,15 @@ class AboutFragment : Fragment() {
             binding.root.findNavController().navigate(R.id.action_aboutFragment_to_licensesFragment)
         }
 
-        binding.textVersionName.text = BuildConfig.VERSION_NAME
+        val buildName = getString(R.string.app_name_suffixed)
+        val buildVersion = NativeLibrary.getBuildVersion()
+        val fullVersionText = "$buildName ($buildVersion)"
+
+        binding.textVersionName.text = fullVersionText
         binding.buttonVersionName.setOnClickListener {
             val clipBoard =
                 requireContext().getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
-            val clip = ClipData.newPlainText(getString(R.string.build), BuildConfig.GIT_HASH)
+            val clip = ClipData.newPlainText(getString(R.string.build), fullVersionText)
             clipBoard.setPrimaryClip(clip)
 
             if (Build.VERSION.SDK_INT < Build.VERSION_CODES.TIRAMISU) {

src/android/app/src/main/jni/native.cpp

@@ -1615,4 +1615,10 @@ JNIEXPORT jstring JNICALL Java_org_yuzu_yuzu_1emu_NativeLibrary_getUpdateUrl(
 }
 #endif
 
+JNIEXPORT jstring JNICALL Java_org_yuzu_yuzu_1emu_NativeLibrary_getBuildVersion(
+        JNIEnv* env,
+        [[maybe_unused]] jobject obj) {
+    return env->NewStringUTF(Common::g_build_version);
+}
+
 } // extern "C"