Skip to content

Conversation

zaferatli
Copy link
Contributor

@zaferatli zaferatli commented Sep 17, 2025

In case of :hermes_enabled flag, pod install warning added

@meta-cla meta-cla bot added the CLA Signed label Sep 17, 2025
Copy link

netlify bot commented Sep 17, 2025

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 0acbbe8
🔍 Latest deploy log https://app.netlify.com/projects/react-native/deploys/68ebb9f7f7f34b00085e24f3
😎 Deploy Preview https://deploy-preview-4807--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Simek Simek requested a review from cipolleschi September 18, 2025 05:29
</TabItem>
</Tabs>

Add `export USE_HERMES=true` to `/ios/.xcode.env` for [Hermes bytecoded](https://github.com/facebook/react-native/blob/4d2e38edd97a87e93cb95a8a0021abf73765cf28/packages/react-native/scripts/react-native-xcode.sh#L166C8-L166C18) bundle on iOS.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you adding this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because on iOS side without this variable release builds is not rendered by hermes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope you should not be needing to add export USE_HERMES=true to your .xcode.env file. I think there is something going on in your setup.

Copy link
Contributor Author

@zaferatli zaferatli Sep 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there is fuzzy situation, when I add hermes_enable value to Podfile I expected to release build would have been byte coded version as same as android without installing pods with USE_HERMES=1 which is not seems on documantation also.

I'm not quite sure but there is fuzy situation which is not explained on this type. We need to add this USE_HERMES=1 bundle exec pod install or mine idk

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hermes is the default engine. If you don't specify anything, that would be used by default.

I think that your setup has some issue and that's not the base situation for all our users, so we should not add this into the documentation

</TabItem>
</Tabs>

Add `export USE_HERMES=true` to `/ios/.xcode.env` for [Hermes bytecoded](https://github.com/facebook/react-native/blob/4d2e38edd97a87e93cb95a8a0021abf73765cf28/packages/react-native/scripts/react-native-xcode.sh#L166C8-L166C18) bundle on iOS.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hermes is the default engine. If you don't specify anything, that would be used by default.

I think that your setup has some issue and that's not the base situation for all our users, so we should not add this into the documentation

@zaferatli
Copy link
Contributor Author

Just to clarify, this flag is only used during the pod installation process. For example, if I change it from true to false, the change won’t take effect until I run pod install again isn't? and this situation is not explicitly informed, I think this case should been added docs, I could add related information in docs if you aggree with me.

@cipolleschi
Copy link
Contributor

@zaferatli correct. That is an env var for Cocoapods only, which controls whether Hermes should be installed.
The thing is, Hermes is the default. So putting HERMES_ENABLED=1 or export USE_HERMES=true is the default behavior.

@zaferatli zaferatli changed the title docs: iOS Hermes export USE_HERMES=true warning added. docs: In case of :hermes_enabled flag, pod install warning added Oct 12, 2025
@zaferatli
Copy link
Contributor Author

@cipolleschi Could you check now? I think we need to inform to pod install required in case of change.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants