fix: build architectures needed only for cpp projects (#753)

okwasniewski · web-flow · commit 95091588167d · 2025-01-28T11:16:47.000Z
### Summary

This PR adds an additional if statement for defining
`reactNativeArchitectures` as its needed only for C++ projects.

### Test plan

CI Green
diff --git a/packages/create-react-native-library/templates/native-common/android/build.gradle b/packages/create-react-native-library/templates/native-common/android/build.gradle
@@ -14,10 +14,12 @@ buildscript {
   }
 }
 
+<% if (project.cpp) { -%>
 def reactNativeArchitectures() {
   def value = rootProject.getProperties().get("reactNativeArchitectures")
   return value ? value.split(",") : ["armeabi-v7a", "x86", "x86_64", "arm64-v8a"]
 }
+<% } -%>
 
 def isNewArchitectureEnabled() {
   return rootProject.hasProperty("newArchEnabled") && rootProject.getProperty("newArchEnabled") == "true"

Original file line number	Diff line number	Diff line change
`@@ -14,10 +14,12 @@ buildscript {`
`14`	`14`	`}`
`15`	`15`	`}`
`16`	`16`
	`17`	`+<% if (project.cpp) { -%>`
`17`	`18`	`def reactNativeArchitectures() {`
`18`	`19`	`def value = rootProject.getProperties().get("reactNativeArchitectures")`
`19`	`20`	`return value ? value.split(",") : ["armeabi-v7a", "x86", "x86_64", "arm64-v8a"]`
`20`	`21`	`}`
	`22`	`+<% } -%>`
`21`	`23`
`22`	`24`	`def isNewArchitectureEnabled() {`
`23`	`25`	`return rootProject.hasProperty("newArchEnabled") && rootProject.getProperty("newArchEnabled") == "true"`