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

fix: Add Hermes to iOS linked libraries #451

Merged
merged 1 commit into from
Nov 2, 2022
Merged

Conversation

Ldoppea
Copy link
Member

@Ldoppea Ldoppea commented Oct 31, 2022

When doing a fresh build from scratch, the app would crash on startup with the following error
Library not loaded: @rpath/hermes.framework/hermes

To fix this we need to add Hermes to project's linked libraries

More info: facebook/react-native#34601 (comment)

When doing a fresh build from scratch, the app would crash on startup
with the following error
`Library not loaded: @rpath/hermes.framework/hermes`

To fix this we need to add Hermes to project's linked libraries and
into embedded libraries

More info: facebook/react-native#34601 (comment)
@Ldoppea Ldoppea force-pushed the fix/hermes_build_link branch from 370cc56 to 801b023 Compare November 2, 2022 10:57
@Ldoppea
Copy link
Member Author

Ldoppea commented Nov 2, 2022

I force-pushed some new config : https://github.com/cozy/cozy-flagship-app/compare/370cc56f243a678bc1f825ff8781c97a15e75cac..801b023e16c6e721f9564f6c462927658d165cf7

Previous edits were not enough to make the app work correctly. They only allowed to run the app on iOS simulators using XCode debug, but the app would still crash on physical device or in simulator without XCode (re-run previous build).

To make it work on every scenario I add to add Hermes to Embedded libraries as stated in facebook/react-native#34601 (comment)

@Crash-- Crash-- self-requested a review November 2, 2022 13:22
@Ldoppea Ldoppea merged commit 9be69c3 into master Nov 2, 2022
@Ldoppea Ldoppea deleted the fix/hermes_build_link branch November 2, 2022 18:02
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