run-ios bug: Try/catch removed from `warnAboutManuallyLinkedLibs.ts` causing exceptions to propagate/crash
See original GitHub issueEnvironment
System:
OS: macOS 11.4
CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
Memory: 921.95 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 16.4.0 - /usr/local/bin/node
Yarn: 1.22.10 - /usr/local/bin/yarn
npm: 7.18.1 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.10.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
Android SDK:
API Levels: 28, 29, 30
Build Tools: 28.0.3, 29.0.3, 30.0.1
System Images: android-30 | Google APIs Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 4.1.2 4.1.2
Xcode: 12.5.1/12E507 - /usr/bin/xcodebuild
Languages:
Java: 11.0.1 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.1 => 17.0.1
react-native: 0.64.1 => 0.64.1
react-native-macos: ^0.63.0 => 0.63.36
npmGlobalPackages:
*react-native*: Not Found
Description
This refers to https://github.com/microsoft/react-native-test-app/issues/375
I was working on moving a community module to use React Native Test App to manage the test app of a community module (https://github.com/react-native-menu/menu), and ran into an error running run-ios
. After investigation, it seems the issue happened upstream beginning in https://github.com/react-native-community/cli/pull/1270 .
A try-catch was removed from packages/platform-ios/src/link/warnAboutManuallyLinkedLibs.ts
, causing the exception to bubble up and crash. This seems like an error, because an Android fix shouldn’t have broken something downstream for iOS.
Reproducible Demo
A little bit contrived since I reproed this using a downstream module, but:
git clone git@github.com:react-native-menu/menu.git
cd menu
mv example/ example2/ # This is to just move the existing test app out of the way temporarily
yarn add react-native-test-app --dev
yarn init-test-app
# The options I chose
# ✔ What is the name of your test app? … MenuExample
# ✔ Which platforms do you need test apps for? › Android, iOS
# ✔ Where should we create the new project? … example
yarn
yarn build:ios
pod install --project-directory=ios
yarn ios
Issue Analytics
- State:
- Created 2 years ago
- Comments:8 (8 by maintainers)
Top Results From Across the Web
Exceptions and debugging - Advanced R. - Hadley Wickham
In this section you'll learn the tools and techniques that help you get to the root cause of an error. You'll learn general...
Read more >try...catch - JavaScript - MDN Web Docs - Mozilla
The try...catch statement is comprised of a try block and either ... Statement that is executed if an exception is thrown in the...
Read more >Exception caught by try/catch block but .Net still crash the ...
For example if I try to delete file or directory that is not exist .Net will caught exception but application will still crash....
Read more >Handling exceptions and errors (Reference) - Prisma
This page covers how to handle exceptions and errors. ... Error handling incomplete in MongoDB. Error handling for the MongoDB connector is incomplete....
Read more >Introduction to error handling in Angular 7: Part 1 - Pusher
As an Angular developer, you already know errors can be caused by ... remove all the operations in the queue until the exception...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Update this is being addressed in https://github.com/react-native-community/cli/commit/596d849d30c07ec056c648afe8d17460afd66d8b#diff-324776bfd410a79f7aaa6bc69a310a8515358fe9abfb81c69ae3744ca784c9fcL45 where I have made Podfile detection the source of truth.
I have a draft PR with a proposed solution ☝️
If you can, please give it a try and let me know how it goes.
cc @grabbou, @thymikee