-
Notifications
You must be signed in to change notification settings - Fork 985
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: make run-ios-device script (#18845)
fixes #16310 We used to reply on `react-native cli` and would pass a `--device` flag to deploy the debug variant of `iOS` app on connected `iPhone`. `react-native cli` under the hood uses `ios-deploy` library to achieve this functionality. This showed many weird issues, specifically in locating connected devices and failures at build step with ambiguous error messages. This commit fixes it by using our custom script `run-ios-devices.sh` which does not rely on `ios-deploy`. We use `libimobiledevice` to identify `UDID` of a connected `iPhone`. We use `xcrun devicectl device install app` and `xcrun devicectl device process launch` to install and launch the app. This works well with `Xcode 15` and `iOS 17.x`. We can now remove `ios-deploy` from `iOS` shell and `nix` overlay. We also set up a logs folder and add a Readme. ## Review notes - connect your iPhone to your Laptop via a cable - `make run-clojure` - `make run-ios-device` (note: no need to pass device name now) ## Platforms - iOS
- Loading branch information
1 parent
50f6c1e
commit 5a7bfc6
Showing
7 changed files
with
81 additions
and
28 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
# Description | ||
|
||
This directory is the destination of logs created during build time of debug builds. | ||
|
||
# Logs | ||
|
||
* `xcrun_device_install.log` - Output from `status-mobile/scripts/run-ios-device.sh`. | ||
- Created by redirecting output of `xcrun simctl install "$UDID" "$APP_PATH"`. | ||
* `xcrun_device_process_launch.log` - Output from `status-mobile/scripts/run-ios-device.sh`. | ||
- Created by specifying `--json-output` flag for `xcrun devicectl device process launch --no-activate --verbose --device "${DEVICE_UUID}" "${INSTALLATION_URL}"`. | ||
* `xcrun_device_process_resume.log` - Output from `status-mobile/scripts/run-ios-device.sh`. | ||
- Created by redirecting output of `xcrun devicectl device process resume --device "${DEVICE_UUID}" --pid "${STATUS_PID}"`. | ||
* `adb_install.log` - Output from `scripts/run-android.sh`. | ||
- Created by redirecting output of `adb install -r ./result/app-debug.apk`. | ||
* `adb_shell_monkey.log` - Output from `status-mobile/scripts/run-android.sh`. | ||
- Created by redirecting output of `adb shell monkey -p im.status.ethereum.debug 1 >`. | ||
* `ios_simulators_list.log` - Output from `status-mobile/scripts/run-ios.sh`. | ||
- Created by redirecting output of `xcrun simctl list devices -j`. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,55 @@ | ||
#!/usr/bin/env bash | ||
set -euo pipefail | ||
set -m # needed to access jobs | ||
|
||
GIT_ROOT=$(cd "${BASH_SOURCE%/*}" && git rev-parse --show-toplevel) | ||
XCRUN_DEVICE_INSTALL_LOG_DIR="${GIT_ROOT}/logs/xcrun_device_install.log" | ||
XCRUN_DEVICE_PROCESS_LAUNCH_LOG_DIR="${GIT_ROOT}/logs/xcrun_device_process_launch.log" | ||
XCRUN_DEVICE_PROCESS_RESUME_LOG_DIR="${GIT_ROOT}/logs/xcrun_device_process_resume.log" | ||
|
||
# Install on the connected device | ||
installAndLaunchApp() { | ||
xcrun devicectl device install app --device "${DEVICE_UUID}" "${APP_PATH}" --json-output "${XCRUN_DEVICE_INSTALL_LOG_DIR}" 2>&1 | ||
|
||
# Extract installationURL | ||
INSTALLATION_URL=$(jq -r '.result.installedApplications[0].installationURL' "${XCRUN_DEVICE_INSTALL_LOG_DIR}") | ||
|
||
# launch the app and put it in background | ||
xcrun devicectl device process launch --no-activate --verbose --device "${DEVICE_UUID}" "${INSTALLATION_URL}" --json-output "${XCRUN_DEVICE_PROCESS_LAUNCH_LOG_DIR}" | ||
|
||
# Extract background PID of status app | ||
STATUS_PID=$(jq -r '.result.process.processIdentifier' "${XCRUN_DEVICE_PROCESS_LAUNCH_LOG_DIR}") | ||
"${GIT_ROOT}/scripts/wait-for-metro-port.sh" 2>&1 | ||
|
||
# now that metro is ready, resume the app from background | ||
xcrun devicectl device process resume --device "${DEVICE_UUID}" --pid "${STATUS_PID}" > "${XCRUN_DEVICE_PROCESS_RESUME_LOG_DIR}" 2>&1 | ||
} | ||
|
||
showXcrunLogs() { | ||
cat "${XCRUN_DEVICE_INSTALL_LOG_DIR}" >&2; | ||
cat "${XCRUN_DEVICE_PROCESS_LAUNCH_LOG_DIR}" >&2; | ||
cat "${XCRUN_DEVICE_PROCESS_RESUME_LOG_DIR}" >&2; | ||
} | ||
|
||
# find the first connected iPhone's UUID | ||
DEVICE_UUID=$(idevice_id -l) | ||
|
||
# Check if any device is connected | ||
if [ -z "${DEVICE_UUID}" ]; then | ||
echo "No connected iPhone device detected." | ||
exit 1 | ||
else | ||
echo "Connected iPhone UDID: ${DEVICE_UUID}" | ||
fi | ||
|
||
BUILD_DIR="${GIT_ROOT}/build" | ||
|
||
#iOS build of debug scheme | ||
xcodebuild -workspace "ios/StatusIm.xcworkspace" -configuration Debug -scheme StatusIm -destination id="${DEVICE_UUID}" -derivedDataPath "${BUILD_DIR}" -verbose | xcbeautify | ||
|
||
APP_PATH="${BUILD_DIR}/Build/Products/Debug-iphoneos/StatusIm.app" | ||
|
||
trap showXcrunLogs EXIT ERR INT QUIT | ||
installAndLaunchApp & | ||
exec "${GIT_ROOT}/scripts/run-metro.sh" 2>&1 | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters