Skip to content

[fix #78] Add dependency on React-Core for fixing RN 0.71.2#77

Closed
ricardoMogg wants to merge 1 commit intopusher:masterfrom
ricardoMogg:react-core
Closed

[fix #78] Add dependency on React-Core for fixing RN 0.71.2#77
ricardoMogg wants to merge 1 commit intopusher:masterfrom
ricardoMogg:react-core

Conversation

@ricardoMogg
Copy link
Copy Markdown

@ricardoMogg ricardoMogg commented Feb 26, 2023

Description

Adding a dependency on React-Core to avoid the Undefined symbol: _RCTEventEmitter error, which seems to be in React-Core.

Screenshot 2023-02-25 at 9 12 39 PM

Issue: #78

CHANGELOG

  • [FIXED] Add React-Core dependency to fix Undefined symbol: _RCTEventEmitter

@ricardoMogg ricardoMogg changed the title Add dependency on React-Core for fixing RN 0.71.2 [Issue #78] Add dependency on React-Core for fixing RN 0.71.2 Feb 26, 2023
@ricardoMogg ricardoMogg changed the title [Issue #78] Add dependency on React-Core for fixing RN 0.71.2 [fix #78] Add dependency on React-Core for fixing RN 0.71.2 Feb 26, 2023
@ricardoMogg
Copy link
Copy Markdown
Author

@benw-pusher

@Keith-wright
Copy link
Copy Markdown
Contributor

Thanks for the contribution. We weren't able to fully verify whether adding React-Core explicitly is still necessary — our current test environment (Xcode 26 + RN 0.70.x) has a separate compatibility issue that prevents us from reaching the linker stage where this symbol error would surface.

That said, the library has shipped through v1.3.4 across multiple React Native versions since this was opened in February 2023, and the _RCTEventEmitter undefined symbol issue hasn't been reported again. In modern React Native, React-Core is included as part of the umbrella pod and an explicit dependency is generally not required.

We're closing this for now on that basis. If anyone is still hitting Undefined symbol: _RCTEventEmitter on a current RN version, please open a new issue with your RN version and Xcode output and we'll investigate.

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.

2 participants