-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix npm run native ios
with xcode12 Command Line Tools
#25532
Conversation
Size Change: 0 B Total Size: 1.17 MB ℹ️ View Unchanged
|
589ebc0
to
5c05789
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried both with Xcode 11 & 12 Command Line Tools and works great! We may need a gutenberg-mobile PR as well though.
@@ -114,7 +114,7 @@ | |||
"test:e2e:bundle:android": "mkdir -p android/app/src/main/assets && npm run rn-bundle -- --reset-cache --platform android --dev false --minify false --entry-file index.js --bundle-output android/app/src/main/assets/index.android.bundle --assets-dest android/app/src/main/res", | |||
"test:e2e:build-app:android": "npm run test:e2e:bundle:android && cd android && ./gradlew clean && ./gradlew assembleDebug", | |||
"test:e2e:install-app:android": "cd android && ./gradlew installDebug", | |||
"test:e2e:bundle:ios": "mkdir -p ios/build/gutenberg/Build/Products/Release-iphonesimulator/GutenbergDemo.app && npm run rn-bundle -- --reset-cache --platform=ios --dev=false --minify false --entry-file=index.js --bundle-output=./ios/build/gutenberg/Build/Products/Release-iphonesimulator/GutenbergDemo.app/main.jsbundle --assets-dest=./ios/build/gutenberg/Build/Products/Release-iphonesimulator/GutenbergDemo.app", | |||
"test:e2e:bundle:ios": "mkdir -p ios/build/gutenberg/Build/Products/Release-iphonesimulator/gutenberg.app && npm run rn-bundle -- --reset-cache --platform=ios --dev=false --minify false --entry-file=index.js --bundle-output=./ios/build/gutenberg/Build/Products/Release-iphonesimulator/gutenberg.app/main.jsbundle --assets-dest=./ios/build/gutenberg/Build/Products/Release-iphonesimulator/gutenberg.app", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we might need the same change in gutenberg-mobile/package.json as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True that!
Made this change here: wordpress-mobile/gutenberg-mobile#2658
Good catch 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚢 Will just trigger CI again to see if iOS build caching works
Fixes wordpress-mobile/gutenberg-mobile#2651
Description
This PR fixes the error:
Print: Entry, ":CFBundleIdentifier", Does Not Exist
when running
npm run native ios
with xcode12 Command Line Tools.The script expects the product to be called
gutenberg.app
, but onproject.pbxproj
it was specified to beGutenbergDemo.app
.Not sure how this was working before, maybe a change on Xcode12 made this error happen.
This is open for better solutions.
How has this been tested?
npm run native ios
.Checklist: