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

Compile the Hermes debug version without asserts #73

Closed
tmikov opened this issue Jul 31, 2019 · 12 comments
Closed

Compile the Hermes debug version without asserts #73

tmikov opened this issue Jul 31, 2019 · 12 comments
Labels
enhancement New feature or request

Comments

@tmikov
Copy link
Contributor

tmikov commented Jul 31, 2019

Duplicating the offline discussion: the Hermes "debug" version is used for RN development and RN debugging, not for debugging Hermes. So, it should be compiled with NDEBUG, to improve performance in the common case.

@scarlac
Copy link

scarlac commented Aug 29, 2019

Is there a workaround that can be applied to existing React Native projects?

@willholen
Copy link
Contributor

This has landed in f72b687. Will publish an NPM soon.

@scarlac
Copy link

scarlac commented Sep 10, 2019

@willholen any chance it will make into a build release of RN or would we have to wait for next major release?

@willholen
Copy link
Contributor

Regardless of which RN version will use 0.2.0 by default, I believe it'll be link compatible with 0.1.1 so a simple npm install .. should allow you to update without touching RN.

@pedropombeiro
Copy link

@willholen any idea when the NPM package containing this fix will get published?

@willholen
Copy link
Contributor

Sorry about the delay! We had some issues with our CI that were only resolved yesterday. The release commit and version bump is already in flight, and is currently in the internal build and regression testing stage. With no other roadblocks, it'll be out today 🤞🤞 (PST).

@willholen
Copy link
Contributor

@pedropombeiro
Copy link

Awesome, thanks a bunch for coming through @willholen!

@willholen
Copy link
Contributor

Celebrated too soon. This is the first fully automated release, and it appears that the .aar files have gotten lost in the process.

@willholen
Copy link
Contributor

As an update, @azundo discovered in #109 that a linker incompatibility had indeed snuck in, which would have prevented some simple upgrades. A fix is currently in flight. Thanks for your patience!

@pedropombeiro
Copy link

Gotcha, thanks for keeping me in the loop!

@willholen
Copy link
Contributor

willholen commented Sep 13, 2019

hermes-engine@0.2.1 has now been released!

Since RN 0.60.5 still uses the previous name hermesvm, anyone trying to manually update with npm install hermes-engine@0.2.1 will need to either
A. symlink hermesvm to hermes-engine in their node_modules
B. add hermesCommand: "../../node_modules/hermes-engine/%OS-BIN%/hermes" to their project.ext.react to use the correct compiler, and similarly replace hermesvm with hermes-engine in their AAR dependencies.

mganandraj pushed a commit to mganandraj/hermes that referenced this issue Jun 22, 2022
## Summary

We want to make progress on passing BinSkim for SDL / security best practices.

Note: after these changes there are still 3 warnings disabled (4146, 4244 and 4267) that may require additional follow-up.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants