Remove unused flag in Old Architecture codegen (facebook#41579)

cipolleschi · facebook-github-bot · commit a6076924bf43 · 2023-11-22T03:50:29.000-08:00
Summary: Pull Request resolved: facebook#41579 This change remove a flag that is unused. ## Changelog [Internal] - Remove unused Flag Reviewed By: dmytrorykun Differential Revision: D51493765 fbshipit-source-id: f8cbce991d80d4f51363cdd4f379e6b214b2b2df
diff --git a/packages/react-native/scripts/react_native_pods_utils/__tests__/script_phases.snap.rb b/packages/react-native/scripts/react_native_pods_utils/__tests__/script_phases.snap.rb
@@ -13,7 +13,6 @@ def snap_get_script_phases_with_codegen_discovery_with_config_file_dir()
     export RCT_SCRIPT_APP_PATH=$RCT_SCRIPT_POD_INSTALLATION_ROOT/
     export RCT_SCRIPT_CONFIG_FILE_DIR=$RCT_SCRIPT_POD_INSTALLATION_ROOT/node_modules
     export RCT_SCRIPT_OUTPUT_DIR=$RCT_SCRIPT_POD_INSTALLATION_ROOT
-    export RCT_SCRIPT_FABRIC_ENABLED=true
     export RCT_SCRIPT_TYPE=withCodegenDiscovery
 
     SCRIPT_PHASES_SCRIPT="$RCT_SCRIPT_RN_DIR/scripts/react_native_pods_utils/script_phases.sh"
@@ -32,7 +31,6 @@ def snap_get_script_phases_with_codegen_discovery_without_config_file_dir()
     export RCT_SCRIPT_APP_PATH=$RCT_SCRIPT_POD_INSTALLATION_ROOT/
     export RCT_SCRIPT_CONFIG_FILE_DIR=
     export RCT_SCRIPT_OUTPUT_DIR=$RCT_SCRIPT_POD_INSTALLATION_ROOT
-    export RCT_SCRIPT_FABRIC_ENABLED=true
     export RCT_SCRIPT_TYPE=withCodegenDiscovery
 
     SCRIPT_PHASES_SCRIPT="$RCT_SCRIPT_RN_DIR/scripts/react_native_pods_utils/script_phases.sh"
diff --git a/packages/react-native/scripts/react_native_pods_utils/script_phases.rb b/packages/react-native/scripts/react_native_pods_utils/script_phases.rb
@@ -13,7 +13,6 @@ def get_script_phases_with_codegen_discovery(options)
         'RCT_SCRIPT_APP_PATH' => "$RCT_SCRIPT_POD_INSTALLATION_ROOT/#{options[:relative_app_root]}",
         'RCT_SCRIPT_CONFIG_FILE_DIR' => "#{options[:relative_config_file_dir] != '' ? "$RCT_SCRIPT_POD_INSTALLATION_ROOT/#{options[:relative_config_file_dir]}" : ''}",
         'RCT_SCRIPT_OUTPUT_DIR' => "$RCT_SCRIPT_POD_INSTALLATION_ROOT",
-        'RCT_SCRIPT_FABRIC_ENABLED' => "#{options[:fabric_enabled]}",
         'RCT_SCRIPT_TYPE' => "withCodegenDiscovery",
     }
     return get_script_template(options[:react_native_path], export_vars)

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,6 @@ def get_script_phases_with_codegen_discovery(options)`
`13`	`13`	`'RCT_SCRIPT_APP_PATH' => "$RCT_SCRIPT_POD_INSTALLATION_ROOT/#{options[:relative_app_root]}",`
`14`	`14`	`'RCT_SCRIPT_CONFIG_FILE_DIR' => "#{options[:relative_config_file_dir] != '' ? "$RCT_SCRIPT_POD_INSTALLATION_ROOT/#{options[:relative_config_file_dir]}" : ''}",`
`15`	`15`	`'RCT_SCRIPT_OUTPUT_DIR' => "$RCT_SCRIPT_POD_INSTALLATION_ROOT",`
`16`		`- 'RCT_SCRIPT_FABRIC_ENABLED' => "#{options[:fabric_enabled]}",`
`17`	`16`	`'RCT_SCRIPT_TYPE' => "withCodegenDiscovery",`
`18`	`17`	`}`
`19`	`18`	`return get_script_template(options[:react_native_path], export_vars)`