Do not remove libjscexecutor.so from release builds (#32932)

Summary: Pull Request resolved: #32932 As the title says, we dont' want to remove `libjscexecutor.so` when baking release builds and having JSC enable as this leads to instacrashes. Fixes #32928 Fixes #32927 Changelog: [Android] [Fixed] - Do not remove libjscexecutor.so from release builds Reviewed By: ShikaSD Differential Revision: D33681932 fbshipit-source-id: 5b59fd1fb76c80c191198d65c916bbbd9232c75b
facebook · Jan 20, 2022 · 574a773 · 574a773
1 parent ceb01db
commit 574a773
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 8 deletions.
diff --git a/packages/react-native-gradle-plugin/src/main/kotlin/com/facebook/react/TaskConfiguration.kt b/packages/react-native-gradle-plugin/src/main/kotlin/com/facebook/react/TaskConfiguration.kt
@@ -240,10 +240,6 @@ private fun Project.cleanupVMFiles(
     } else {
       // For JSC, delete all the libhermes* files
       it.include("**/libhermes*.so")
-      // Delete the libjscexecutor from release build
-      if (cleanup) {
-        it.include("**/libjscexecutor.so")
-      }
     }
   }
       .visit { visit ->

diff --git a/react.gradle b/react.gradle
@@ -372,10 +372,6 @@ afterEvaluate {
                 } else {
                     // For JSC, delete all the libhermes* files
                     include "**/libhermes*.so"
-                    // Delete the libjscexecutor from release build
-                    if (cleanup) {
-                        include "**/libjscexecutor.so"
-                    }
                 }
             }.visit { details ->
                 def targetVariant1 = ".*/transforms/[^/]*/${variant.name}/.*"