fix it

Author: silbinarywolf

src/androidbuild/Apk.zig

@@ -460,18 +460,6 @@ fn doInstallApk(apk: *Apk) Allocator.Error!*Step.InstallFile {
             @panic(b.fmt("artifact[{d}] has no 'target' set", .{artifact_index}));
         };
 
-        // NOTE(jae): 2026-02-01
-        // If not explicitly set in users build, default to using LLVM and LLD for Android builds
-        // as that's the same toolchain that the Android SDK uses.
-        //
-        // This also can resolve issues with Zigs linker not yet supporting certain compression schemes/etc
-        if (artifact.use_llvm == null) {
-            artifact.use_llvm = true;
-        }
-        if (artifact.use_lld == null) {
-            artifact.use_lld = true;
-        }
-
         // https://developer.android.com/ndk/guides/abis#native-code-in-app-packages
         const so_dir: []const u8 = switch (target.result.cpu.arch) {
             .aarch64 => "arm64-v8a",
@@ -530,18 +518,14 @@ fn doInstallApk(apk: *Apk) Allocator.Error!*Step.InstallFile {
         // - To know where C header /lib files are via setLibCFile and linkLibC
         // - Provide path to additional libraries to link to
         {
-            if (artifact.linkage) |linkage| {
-                if (linkage == .dynamic) {
-                    updateSharedLibraryOptions(artifact);
-                }
+            if (artifact.root_module.link_libc == null) {
+                artifact.root_module.link_libc = true;
             }
-            apk.setLibCFile(artifact);
-            apk.addLibraryPaths(artifact.root_module);
-            artifact.root_module.link_libc = true;
+            apk.updateArtifact(artifact, so_dir, apk_files);
 
             // Apply workaround for Zig 0.14.0 stable
             //
-            // This *must* occur after "apk.updateLinkObjects" for the root package otherwise
+            // This *must* occur after apk.updateArtifact (apk.updateLinkObjects) for the root package otherwise
             // you may get an error like: "unable to find dynamic system library 'c++abi_zig_workaround'"
             apk.applyLibLinkCppWorkaroundIssue19(artifact);
         }
@@ -787,33 +771,55 @@ fn setLibCFile(apk: *Apk, compile: *Step.Compile) void {
     compile.setLibCFile(android_libc_path);
 }
 
+fn updateArtifact(apk: *Apk, artifact: *Step.Compile, so_dir: []const u8, raw_top_level_apk_files: *Step.WriteFile) void {
+    // If you have a library that is being built as an *.so then install it
+    // alongside your library.
+    //
+    // This was initially added to support building SDL2 with Zig.
+    if (artifact.linkage) |linkage| {
+        if (linkage == .dynamic) {
+            updateSharedLibraryOptions(artifact);
+            _ = raw_top_level_apk_files.addCopyFile(artifact.getEmittedBin(), apk.b.fmt("lib/{s}/lib{s}.so", .{ so_dir, artifact.name }));
+        }
+    }
+
+    // NOTE(jae): 2026-02-01
+    // If not explicitly set in users build, default to using LLVM and LLD for Android builds
+    // as that's the same toolchain that the Android SDK uses.
+    //
+    // This also can resolve issues with Zigs linker not yet supporting certain compression schemes/etc
+    if (artifact.use_lld == null) {
+        artifact.use_lld = true;
+    }
+    if (artifact.use_llvm == null) {
+        artifact.use_llvm = true;
+    }
+
+    // NOTE(jae): 2026-04-12
+    // Android should default to using fPIC to avoid compilation issues.
+    if (artifact.root_module.pic == null) {
+        artifact.root_module.pic = true;
+    }
+
+    // If library is built using C or C++ then setLibCFile
+    if (artifact.root_module.link_libc == true or
+        artifact.root_module.link_libcpp == true)
+    {
+        apk.setLibCFile(artifact);
+    }
+
+    // Add library paths to find "android", "log", etc
+    apk.addLibraryPaths(artifact.root_module);
+}
+
 fn updateLinkObjects(apk: *Apk, root_artifact: *Step.Compile, so_dir: []const u8, raw_top_level_apk_files: *Step.WriteFile) void {
-    const b = apk.b;
     for (root_artifact.root_module.link_objects.items) |link_object| {
         switch (link_object) {
             .other_step => |artifact| {
                 switch (artifact.kind) {
                     .lib => {
-                        // If you have a library that is being built as an *.so then install it
-                        // alongside your library.
-                        //
-                        // This was initially added to support building SDL2 with Zig.
-                        if (artifact.linkage) |linkage| {
-                            if (linkage == .dynamic) {
-                                updateSharedLibraryOptions(artifact);
-                                _ = raw_top_level_apk_files.addCopyFile(artifact.getEmittedBin(), b.fmt("lib/{s}/lib{s}.so", .{ so_dir, artifact.name }));
-                            }
-                        }
-
-                        // If library is built using C or C++ then setLibCFile
-                        if (artifact.root_module.link_libc == true or
-                            artifact.root_module.link_libcpp == true)
-                        {
-                            apk.setLibCFile(artifact);
-                        }
-
-                        // Add library paths to find "android", "log", etc
-                        apk.addLibraryPaths(artifact.root_module);
+                        // Update updateSharedLibraryOptions, libCFile, addLibraryPaths
+                        apk.updateArtifact(artifact, so_dir, raw_top_level_apk_files);
 
                         // Update libraries linked to this library
                         apk.updateLinkObjects(artifact, so_dir, raw_top_level_apk_files);