Skip to content

Conversation

@tomekzaw
Copy link
Member

@tomekzaw tomekzaw commented Jul 11, 2025

Summary

This PR includes react-native-flags.cmakeand calls target_compile_reactnative_options in CMakeLists.txt which sets necessary flags and definitions instead of setting them manually.

Test plan

@tomekzaw tomekzaw requested review from piaskowyk and tjzel July 11, 2025 09:03
@tomekzaw tomekzaw changed the title Use react-native-flags.cmake Set compile options using target_compile_reactnative_options from react-native-flags.cmake Jul 11, 2025
@tomekzaw tomekzaw added this pull request to the merge queue Jul 14, 2025
Merged via the queue into main with commit 7c13ee6 Jul 14, 2025
11 checks passed
@tomekzaw tomekzaw deleted the @tomekzaw/react-native-flags-cmake branch July 14, 2025 08:20
jakex7 added a commit to software-mansion/react-native-svg that referenced this pull request Jul 14, 2025
…2710)

# Summary

PR based on
software-mansion/react-native-reanimated#7776 by
@tomekzaw.

To support `[email protected]`, we need to include
`RN_SERIALIZABLE_STATE`. This can be done using
`target_compile_reactnative_options`, which sets the necessary flags and
definitions automatically, instead of setting them manually.

## Test Plan

CI is green and `tests-example` compiles on `[email protected]`.
github-merge-queue bot pushed a commit that referenced this pull request Aug 16, 2025
## Summary

Work in progress.

If you're getting `TypeError: (0 , _util.styleText) is not a function`
during `yarn start`, please use Node.js 20 instead by running `nvm use
20`.

`RN_SERIALIZABLE_STATE`:
https://github.com/facebook/react-native/pulls?q=sort%3Aupdated-desc+is%3Apr+rn_serializable_state+is%3Aclosed
* Merged separately in
#7776

## Test plan

|| Fabric (iOS) | Fabric (Android) | Web | MacOS | TVOS (iOS) | TVOS
(Android) |
|-|-|-|-|-|-|-|
| Builds | ✅ | ✅ | ✅ | not upgraded | ✅ (not upgraded) | ✅ (not
upgraded)
| Runs | ✅ | ✅ | ✅ | not upgraded | ✅ (not upgraded) | ✅ (not upgraded)

---------

Co-authored-by: Mateusz Łopaciński <[email protected]>
Co-authored-by: Sebastian Kuśnierz <[email protected]>
skusnierz pushed a commit that referenced this pull request Sep 30, 2025
…react-native-flags.cmake` (#7776)

## Summary

This PR includes `react-native-flags.cmake`and calls
`target_compile_reactnative_options` in CMakeLists.txt which sets
necessary flags and definitions instead of setting them manually.

## Test plan
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants