Skip to content

Commit ce06214

Browse files
hramosfacebook-github-bot
authored andcommitted
Fix shellcheck warnings in Hermes build scripts
Summary: Fixes some shellcheck warnings that arose in D36295406 (ae28880) when the Hermes build scripts were copied over. Changelog: [Internal] Reviewed By: cortinico Differential Revision: D36321074 fbshipit-source-id: 0eae36a4f793daa0e12786ba7989d4eac6b2e256
1 parent 58bc981 commit ce06214

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

sdks/hermes-engine/utils/build-apple-framework.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,9 +90,9 @@ function create_universal_framework {
9090
done
9191

9292
mkdir universal
93-
xcodebuild -create-xcframework $args -output "universal/hermes.xcframework"
93+
xcodebuild -create-xcframework "$args" -output "universal/hermes.xcframework"
9494

95-
for platform in $@; do
95+
for platform in "$@"; do
9696
rm -r "$platform"
9797
done
9898

sdks/hermes-engine/utils/build-ios-framework.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
# This source code is licensed under the MIT license found in the
55
# LICENSE file in the root directory of this source tree.
66

7+
# shellcheck source=xplat/js/react-native-github/sdks/hermes-engine/utils/build-apple-framework.sh
78
. ./utils/build-apple-framework.sh
89

910
if [ ! -d destroot/Library/Frameworks/universal/hermes.xcframework ]; then

sdks/hermes-engine/utils/build-mac-framework.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@
44
# This source code is licensed under the MIT license found in the
55
# LICENSE file in the root directory of this source tree.
66

7+
# shellcheck source=xplat/js/react-native-github/sdks/hermes-engine/utils/build-apple-framework.sh
78
. ./utils/build-apple-framework.sh
9+
810
if [ ! -d destroot/Library/Frameworks/macosx/hermes.framework ]; then
911
mac_deployment_target=$(get_mac_deployment_target)
1012

0 commit comments

Comments
 (0)