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

Enable hermes debugger on all build variants #33

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

Kudo
Copy link

@Kudo Kudo commented Jan 22, 2024

Why

close ENG-11034

How

patch react-native to set HERMES_ENABLE_DEBUGGER for all variants

Copy link

linear bot commented Jan 22, 2024

@Kudo Kudo merged commit 5aa6b72 into exp-latest Jan 22, 2024
2 checks passed
@Kudo Kudo deleted the @kudo/hermes-enable-debugger branch January 22, 2024 15:13
Kudo added a commit to expo/expo that referenced this pull request Jan 22, 2024
# Why

fixed #26044
close ENG-11034

# How

we need to enable hermes debugging feature even on expo-go release
build. previously we did this when generating versioned hermes. since we
now use unversioned variant to serve sdk 50, we should change our way to
enable the hermes debugging feature.

this pr with expo/react-native#33 are to enable
the `HERMES_ENABLE_DEBUGGER` for all variants
note that: this pr would also increase ios build time because we now
building hermes from source.
Kudo added a commit to expo/expo that referenced this pull request Jan 22, 2024
fixed #26044
close ENG-11034

we need to enable hermes debugging feature even on expo-go release
build. previously we did this when generating versioned hermes. since we
now use unversioned variant to serve sdk 50, we should change our way to
enable the hermes debugging feature.

this pr with expo/react-native#33 are to enable
the `HERMES_ENABLE_DEBUGGER` for all variants
note that: this pr would also increase ios build time because we now
building hermes from source.
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.

3 participants