Commit 08ef7b1
Fix React-RCTAnimatedModuleProvider build failures (#57252)
Summary:
`React-RCTAnimatedModuleProvider` could fail to build because of two issues in its podspec:
- **Missing space between compiler flags**: `new_arch_enabled_flag` and `js_engine_flags()` were concatenated without a separator, producing a single malformed flag (e.g. `-DRCT_NEW_ARCH_ENABLED=1-DUSE_HERMES=1`) instead of two distinct flags.
- **Missing Yoga dependency**: the pod uses Yoga headers (through `React-Fabric/animated`) but did not declare the `Yoga` dependency nor its private header search path, leading to a `'yoga/...' file not found` build error.
This adds the missing space, declares the `Yoga` dependency, and adds `$(PODS_ROOT)/Headers/Private/Yoga` to the header search paths.
## Changelog:
[IOS] [FIXED] - Fix React-RCTAnimatedModuleProvider build by adding the missing Yoga dependency and a missing space between compiler flags
Pull Request resolved: #57252
Test Plan:
Without these changes, building the pod fails with errors such as:
```
fatal error: 'yoga/Yoga.h' file not found
```
and the macro flags being mangled into a single unrecognized define.
- Run `pod install` and build the app on iOS.
- Confirm `React-RCTAnimatedModuleProvider` compiles and the build succeeds.
Reviewed By: zeyap
Differential Revision: D108885720
Pulled By: cortinico
fbshipit-source-id: b527570857d704b802382334a413ab15879ba13a1 parent bc20ec8 commit 08ef7b1
1 file changed
Lines changed: 3 additions & 1 deletion
File tree
- packages/react-native/ReactApple/RCTAnimatedModuleProvider
packages/react-native/ReactApple/RCTAnimatedModuleProvider/React-RCTAnimatedModuleProvider.podspec
Lines changed: 3 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| 26 | + | |
26 | 27 | | |
27 | 28 | | |
28 | 29 | | |
| |||
52 | 53 | | |
53 | 54 | | |
54 | 55 | | |
| 56 | + | |
55 | 57 | | |
56 | 58 | | |
57 | 59 | | |
| |||
0 commit comments