Modified: branches/safari-609-branch/Tools/ChangeLog (262245 => 262246)
--- branches/safari-609-branch/Tools/ChangeLog 2020-05-28 16:56:10 UTC (rev 262245)
+++ branches/safari-609-branch/Tools/ChangeLog 2020-05-28 17:16:31 UTC (rev 262246)
@@ -1,3 +1,35 @@
+2020-05-28 Ryan Haddad <ryanhad...@apple.com>
+
+ Cherry-pick r262143. rdar://problem/63517635
+
+ webkitpy: simctl list may have stderr logging
+ https://bugs.webkit.org/show_bug.cgi?id=212376
+ <rdar://problem/63517635>
+
+ Unreviewed infrastructure fix.
+
+ * Scripts/webkitpy/xcode/simulated_device.py:
+ (SimulatedDeviceManager.populate_available_devices): Only parse stdout, log error when
+ json decoding fails.
+ (SimulatedDevice.is_usable): Only parse stdout.
+ (SimulatedDevice.launch_app): Ditto.
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@262143 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2020-05-26 Jonathan Bedard <jbed...@apple.com>
+
+ webkitpy: simctl list may have stderr logging
+ https://bugs.webkit.org/show_bug.cgi?id=212376
+ <rdar://problem/63517635>
+
+ Unreviewed infrastructure fix.
+
+ * Scripts/webkitpy/xcode/simulated_device.py:
+ (SimulatedDeviceManager.populate_available_devices): Only parse stdout, log error when
+ json decoding fails.
+ (SimulatedDevice.is_usable): Only parse stdout.
+ (SimulatedDevice.launch_app): Ditto.
+
2020-05-18 Ryan Haddad <ryanhad...@apple.com>
Unreviewed test gardening for rdar://59610140 and rdar://59611168.
Modified: branches/safari-609-branch/Tools/Scripts/webkitpy/xcode/simulated_device.py (262245 => 262246)
--- branches/safari-609-branch/Tools/Scripts/webkitpy/xcode/simulated_device.py 2020-05-28 16:56:10 UTC (rev 262245)
+++ branches/safari-609-branch/Tools/Scripts/webkitpy/xcode/simulated_device.py 2020-05-28 17:16:31 UTC (rev 262246)
@@ -129,8 +129,9 @@
return
try:
- simctl_json = json.loads(host.executive.run_command([SimulatedDeviceManager.xcrun, 'simctl', 'list', '--json'], decode_output=False))
+ simctl_json = json.loads(host.executive.run_command([SimulatedDeviceManager.xcrun, 'simctl', 'list', '--json'], decode_output=False, return_stderr=False))
except (ValueError, ScriptError):
+ _log.error('Failed to decode json output')
return
SimulatedDeviceManager._device_identifier_to_name = {device['identifier']: device['name'] for device in simctl_json['devicetypes']}
@@ -563,7 +564,7 @@
_log.debug(u'{} has no service to check if the device is usable'.format(self.device_type.software_variant))
return True
- system_processes = self.executive.run_command([SimulatedDeviceManager.xcrun, 'simctl', 'spawn', self.udid, 'launchctl', 'print', 'system'], decode_output=False)
+ system_processes = self.executive.run_command([SimulatedDeviceManager.xcrun, 'simctl', 'spawn', self.udid, 'launchctl', 'print', 'system'], decode_output=False, return_stderr=False)
if re.search(r'"{}"'.format(home_screen_service), system_processes) or re.search(r'A\s+{}'.format(home_screen_service), system_processes):
return True
return False
@@ -638,6 +639,7 @@
['xcrun', 'simctl', 'launch', self.udid, bundle_id] + args,
env=environment_to_use,
error_handler=_log_debug_error,
+ return_stderr=False,
)
match = re.match(r'(?P<bundle>[^:]+): (?P<pid>\d+)\n', output)
# FIXME: We shouldn't need to check the PID <rdar://problem/31154075>.