Skip to content
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

react-native-reanimated 3.9.0-rc.1 fails to build on iOS with new architecture enabled in Release build #5912

Closed
timbocole opened this issue Apr 18, 2024 · 3 comments · Fixed by #5925
Assignees
Labels
Missing repro This issue need minimum repro scenario Platform: iOS This issue is specific to iOS

Comments

@timbocole
Copy link

Description

When building an application for iOS with new architecture enabled, with a non-release build, compilation fails with:

❌  app/node_modules/react-native-reanimated/apple/REAModule.mm:179:9: no visible @interface for 'REAModule' declares the selector 'setReaSurfacePresenter'

  [self setReaSurfacePresenter];

This is due to #5901 removing the conditional #ifndef NDEBUG around the invocation of setReaSurfacePresenter, however the definition of this api is still wrapped in a conditional#ifndef NDEBUG block.

Reapplying the conditional to the invocation resolves the problem however I am unsure of expected behaviour here.

Steps to reproduce

Attempt to build a NON-RELEASE app for iOS using new architecture.

Snack or a link to a repository

NONE

Reanimated version

3.9.0-rc.1

React Native version

0.74.0-rc.9

Platforms

iOS

JavaScript runtime

Hermes

Workflow

React Native

Architecture

Fabric (New Architecture)

Build type

Debug app & dev bundle

Device

None

Device model

No response

Acknowledgements

Yes

@github-actions github-actions bot added the Platform: iOS This issue is specific to iOS label Apr 18, 2024
Copy link

Hey! 👋

The issue doesn't seem to contain a minimal reproduction.

Could you provide a snack or a link to a GitHub repository under your username that reproduces the problem?

@github-actions github-actions bot added the Missing repro This issue need minimum repro scenario label Apr 18, 2024
@piaskowyk piaskowyk self-assigned this Apr 22, 2024
@timbocole timbocole changed the title react-native-reanimated 3.9.0-rc.1 fails to build on iOS with new architecture enabled in non Release build react-native-reanimated 3.9.0-rc.1 fails to build on iOS with new architecture enabled in Release build Apr 22, 2024
@piaskowyk
Copy link
Member

@timbocole thanks a lot for the report 🙌

@timbocole
Copy link
Author

timbocole commented Apr 22, 2024

@timbocole thanks a lot for the report 🙌

Thanks for the fix - I can see now that I totally inverted the report though as was actually using a Release config without realising and got thrown by the double negative in the code 😂

github-merge-queue bot pushed a commit that referenced this issue Apr 23, 2024
## Summary

This PR fixes production build for iOS. In
#5901 I
called `setReaSurfacePresenter` method but this one was defined only on
debug build.

Fixes
#5912

## Test plan

Run production build for iOS
piaskowyk added a commit that referenced this issue Apr 26, 2024
## Summary

This PR fixes production build for iOS. In
#5901 I
called `setReaSurfacePresenter` method but this one was defined only on
debug build.

Fixes
#5912

## Test plan

Run production build for iOS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Missing repro This issue need minimum repro scenario Platform: iOS This issue is specific to iOS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants