Skip to content

Commit 0deb436

Browse files
cipolleschimeta-codesync[bot]
authored andcommitted
Update stale 'Hermes V1' comments (#56757)
Summary: Pull Request resolved: #56757 Update comments that still say 'Hermes V1' as if it is an opt-in feature. Since Hermes (formerly called Hermes V1) is now the only supported engine: - hermes-engine.podspec: drop 'when using Hermes V1' from hermesc note - PathUtils.kt: rewrite the hermesc path comment to drop the 'opted in to Hermes V1' phrasing - hermes-engine/build.gradle.kts: 'Hermes V1 by default...' -> 'Hermes by default...' Differential Revision: D104649634
1 parent e3f5814 commit 0deb436

3 files changed

Lines changed: 5 additions & 7 deletions

File tree

packages/gradle-plugin/react-native-gradle-plugin/src/main/kotlin/com/facebook/react/utils/PathUtils.kt

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -123,11 +123,9 @@ private fun detectCliFile(
123123
* 1. The path provided by the `hermesCommand` config in the `react` Gradle extension
124124
* 2. The file located in `node_modules/react-native/sdks/hermes/build/bin/hermesc`. This will be
125125
* used if the user is building Hermes from source.
126-
* 3. The file located in `node_modules/react-native/sdks/hermesc/%OS-BIN%/hermesc` where `%OS-BIN%`
127-
* is substituted with the correct OS arch. This will be used if the user is using a precompiled
128-
* hermes-engine package. Or, if the user has opted in to use Hermes V1, the used file will be
129-
* located in `node_modules/hermes-compiler/%OS-BIN%/hermesc` where `%OS-BIN%` is substituted
130-
* with the correct OS arch.
126+
* 3. The file located in `node_modules/hermes-compiler/%OS-BIN%/hermesc` where `%OS-BIN%` is
127+
* substituted with the correct OS arch. This is used when Hermes is consumed as a prebuilt
128+
* package via the `hermes-compiler` npm package.
131129
* 4. Fails otherwise
132130
*/
133131
internal fun detectOSAwareHermesCommand(

packages/react-native/ReactAndroid/hermes-engine/build.gradle.kts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -237,7 +237,7 @@ val buildHermesLib by
237237
}
238238

239239
// The repeated tasks below named "*WithDebugger" are required by Fantom.
240-
// Hermes V1 by default builds with the debugger disabled, while Fantom needs
240+
// Hermes by default builds with the debugger disabled, while Fantom needs
241241
// it to be enabled as it does a debug build of React Native.
242242
val configureBuildForHermesWithDebugger by
243243
tasks.registering(CustomExecTask::class) {

packages/react-native/sdks/hermes-engine/hermes-engine.podspec

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ Pod::Spec.new do |spec|
6565
end
6666

6767
# When using the local prebuilt tarball, it should include hermesc compatible with the used VM.
68-
# In other cases, when using Hermes V1, the prebuilt versioned binaries can be used.
68+
# In other cases, the prebuilt versioned hermesc binaries from the hermes-compiler npm package can be used.
6969
if source_type != HermesEngineSourceType::LOCAL_PREBUILT_TARBALL
7070
hermes_compiler_path = File.dirname(Pod::Executable.execute_command('node', ['-p',
7171
"require.resolve(\"hermes-compiler\", {paths: [\"#{react_native_path}\"]})", __dir__]).strip

0 commit comments

Comments
 (0)