mirror of
https://github.com/facebook/react-native.git
synced 2025-11-01 09:14:26 +00:00
Stabilize Hermes build in CI (#36137)
Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/36137 This change stabilize the build for Hermes in CI, making it more robust in case the rename step fails or the wrong params are passed to hermes. ## Changelog: [internal] - Update Hermes build logic Reviewed By: cortinico Differential Revision: D43232941 fbshipit-source-id: 1121716c69dd3ca247ffeaed57e0d4dfd5daca84
This commit is contained in:
committed by
Facebook GitHub Bot
parent
7f20deb428
commit
12cd42d2fa
@@ -131,17 +131,19 @@ function create_universal_framework {
|
||||
for i in "${!platforms[@]}"; do
|
||||
local platform="${platforms[$i]}"
|
||||
local hermes_framework_path="${platform}/hermes.framework"
|
||||
local dSYM_path="${platform}/hermes.framework"
|
||||
local dSYM_path="$hermes_framework_path"
|
||||
local dSYM_base_path="$HERMES_PATH/destroot/Library/Frameworks"
|
||||
|
||||
# TODO: remove this when the hermes team fixes the dSYM issue
|
||||
if [[ "$platform" == "catalyst" ]]; then
|
||||
# If the dSYM rename has failed, the dSYM are generated as 0.dSYM
|
||||
# (Apple default name) rather then hermes.framework.dSYM.
|
||||
if [[ -e "$dSYM_base_path/${platform}/0.dSYM" ]]; then
|
||||
dSYM_path="${platform}/0"
|
||||
fi
|
||||
|
||||
args+="-framework $hermes_framework_path "
|
||||
|
||||
# Path to dSYM must be absolute
|
||||
args+="-debug-symbols $HERMES_PATH/destroot/Library/Frameworks/$dSYM_path.dSYM "
|
||||
args+="-debug-symbols $dSYM_base_path/$dSYM_path.dSYM "
|
||||
done
|
||||
|
||||
mkdir -p universal
|
||||
|
||||
Reference in New Issue
Block a user