fix(android): revert "always include ReactNative-application.cmake …

…on 0.76" (#2341) This reverts commit c1a56a7.
microsoft · Dec 10, 2024 · 45eabdb · 45eabdb
1 parent 0eaf227
commit 45eabdb
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 14 deletions.
diff --git a/android/app/build.gradle b/android/app/build.gradle
@@ -108,7 +108,7 @@ android {
         resValue "string", "app_name", project.ext.react.appName
 
         def cppStd = reactNativeVersion >= v(0, 74, 0) ? "-std=c++20" : "-std=c++17"
-        if (reactNativeVersion >= v(0, 76, 0) || enableNewArchitecture) {
+        if (enableNewArchitecture) {
             externalNativeBuild {
                 cmake {
                     arguments "-DANDROID_STL=c++_shared",

diff --git a/android/app/src/main/jni/AutolinkingCompat.h b/android/app/src/main/jni/AutolinkingCompat.h
@@ -9,18 +9,14 @@
 #define autolinking_cxxModuleProvider facebook::react::autolinking_cxxModuleProvider
 #define autolinking_registerProviders facebook::react::autolinking_registerProviders
 
-#elif __has_include(<rncli.h>)  // < 0.75
+#else  // < 0.75
 
 #include <rncli.h>
 
 #define autolinking_ModuleProvider facebook::react::rncli_ModuleProvider
 #define autolinking_cxxModuleProvider facebook::react::rncli_cxxModuleProvider
 #define autolinking_registerProviders facebook::react::rncli_registerProviders
 
-#else  // Autolinking is handled elsewhere in the old architecture (paper)
-
-#define REACTAPP_LEGACY_AUTOLINKING 1
-
 #endif  // __has_include(<autolinking.h>)
 
 #endif  // REACTAPP_JNI_AUTOLINKINGCOMPAT_H_
diff --git a/android/app/src/main/jni/ComponentsRegistry.cpp b/android/app/src/main/jni/ComponentsRegistry.cpp
@@ -32,10 +32,8 @@ void ComponentsRegistry::registerNatives()
 
     facebook::react::DefaultComponentsRegistry::registerComponentDescriptorsFromEntryPoint =
         [](std::shared_ptr<ComponentDescriptorProviderRegistry const> registry) {
-#ifndef REACTAPP_LEGACY_AUTOLINKING
             // Register providers generated by `@react-native-community/cli`
             autolinking_registerProviders(registry);
-#endif  // !REACTAPP_LEGACY_AUTOLINKING
         };
 }
 

diff --git a/android/app/src/main/jni/OnLoad.cpp b/android/app/src/main/jni/OnLoad.cpp
@@ -19,7 +19,6 @@ using facebook::react::TurboModule;
 
 namespace
 {
-#ifndef REACTAPP_LEGACY_AUTOLINKING
     std::shared_ptr<TurboModule> cxxModuleProvider(const std::string &name,
                                                    const std::shared_ptr<CallInvoker> &jsInvoker)
     {
@@ -43,18 +42,15 @@ namespace
         // And we fallback to the module providers autolinked by RN CLI
         return autolinking_ModuleProvider(name, params);
     }
-#endif  // !REACTAPP_LEGACY_AUTOLINKING
 }  // namespace
 
 JNIEXPORT jint JNICALL JNI_OnLoad(JavaVM *vm, void *)
 {
     return facebook::jni::initialize(vm, [] {
-#ifndef REACTAPP_LEGACY_AUTOLINKING
         DefaultTurboModuleManagerDelegate::cxxModuleProvider = &cxxModuleProvider;
         DefaultTurboModuleManagerDelegate::javaModuleProvider = &javaModuleProvider;
         DefaultComponentsRegistry::registerComponentDescriptorsFromEntryPoint =
             &autolinking_registerProviders;
-#endif  // !REACTAPP_LEGACY_AUTOLINKING
     });
 }
 

diff --git a/android/app/src/main/jni/TurboModuleManagerDelegate.cpp b/android/app/src/main/jni/TurboModuleManagerDelegate.cpp
@@ -28,13 +28,11 @@ std::shared_ptr<TurboModule> TurboModuleManagerDelegate::getTurboModule(StringRe
 std::shared_ptr<TurboModule> TurboModuleManagerDelegate::getTurboModule(
     StringRef name, const JavaTurboModule::InitParams &params)
 {
-#ifndef REACTAPP_LEGACY_AUTOLINKING
     // Try autolinked module providers first
     auto module = autolinking_ModuleProvider(name, params);
     if (module != nullptr) {
         return module;
     }
-#endif  // !REACTAPP_LEGACY_AUTOLINKING
 
     return rncore_ModuleProvider(name, params);
 }