Skip to content

Commit b7349f9

Browse files
RSNarafacebook-github-bot
authored andcommitted
Use "Legacy Build System"
Summary: React Native needs to be updated to support XCode 10 (GitHub issue: [19573](https://github.com/facebook/react-nåative/issues/19573)). Until we make that change, our CI needs to rely on the legacy build system in XCode 10 to build `RNTester`, as opposed to the new build system made default in XCode 10. This diff uses the `-UseModernBuildSystem=NO` `xcodebuild` flag to enforce that requirement. **Note:** I did a search within `react-native-github/scripts` for `xcodebuild` and passed in the `-UseModernBuildSystem=NO` flag. I figured that every time we use `xcodebuild`, we should enforce the build system. � This should hopefully fix the `react-native-oss-ios` test. Reviewed By: hramos Differential Revision: D10456139 fbshipit-source-id: 795f32b3ceba6a5a24ab6ccbf4f5160d42746801
1 parent 99471f8 commit b7349f9

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

scripts/objc-test.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,7 @@ xcodebuild \
8383
-scheme $SCHEME \
8484
-sdk $SDK \
8585
-destination "$DESTINATION" \
86+
-UseModernBuildSystem=NO \
8687
build test \
8788
| xcpretty --report junit --output "$HOME/react-native/reports/junit/$TEST_NAME/results.xml"
8889

@@ -93,6 +94,7 @@ xcodebuild \
9394
-project "RNTester/RNTester.xcodeproj" \
9495
-scheme $SCHEME \
9596
-sdk $SDK \
97+
-UseModernBuildSystem=NO \
9698
build
9799

98100
fi

scripts/run-ci-e2e-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,7 @@ try {
216216
`"${scheme}"`,
217217
'-sdk',
218218
sdk,
219+
'-UseModernBuildSystem=NO',
219220
'test',
220221
].join(' ') +
221222
' | ' +

0 commit comments

Comments
 (0)