Diff
Modified: trunk/Tools/ChangeLog (285778 => 285779)
--- trunk/Tools/ChangeLog 2021-11-13 21:57:32 UTC (rev 285778)
+++ trunk/Tools/ChangeLog 2021-11-13 21:58:50 UTC (rev 285779)
@@ -1,3 +1,15 @@
+2021-11-13 Jonathan Bedard <jbed...@apple.com>
+
+ Unreviewed, reverting r285772.
+
+ Ownership issues in some XCode installs
+
+ Reverted changeset:
+
+ "[webkitpy] Symlink daemons into simulator runtime root"
+ https://bugs.webkit.org/show_bug.cgi?id=233080
+ https://commits.webkit.org/r285772
+
2021-11-13 Tyler Wilcock <tyle...@apple.com>
AX: Make accessibility/mac/header.html test async to fix it for isolated tree mode
Modified: trunk/Tools/Scripts/webkitpy/api_tests/manager.py (285778 => 285779)
--- trunk/Tools/Scripts/webkitpy/api_tests/manager.py 2021-11-13 21:57:32 UTC (rev 285778)
+++ trunk/Tools/Scripts/webkitpy/api_tests/manager.py 2021-11-13 21:58:50 UTC (rev 285779)
@@ -152,22 +152,6 @@
def _initialize_devices(self):
if 'simulator' in self._port.port_name:
SimulatedDeviceManager.initialize_devices(DeviceRequest(self._port.DEVICE_TYPE, allow_incomplete_match=True), self.host, simulator_ui=False)
-
- # A Daemons executable path must be located within the runtime root.
- roots = {
- device.platform_device.runtime.root for device in SimulatedDeviceManager.INITIALIZED_DEVICES
- if device.platform_device.runtime and device.platform_device.runtime.root
- }
- fs = self._port.host.filesystem
- for root in roots:
- _log.debug("Linking Daemons into runtime root '{}'".format(root))
- for file in fs.files_under(self._port.path_to_daemons()):
- target = fs.join(root, 'usr', 'local', 'bin', 'webkit-testing', fs.basename(file))
- fs.maybe_make_directory(fs.dirname(target))
- if fs.isfile(target):
- fs.remove(target)
- fs.symlink(file, target)
-
elif 'device' in self._port.port_name:
raise RuntimeError('Running api tests on {} is not supported'.format(self._port.port_name))
Modified: trunk/Tools/Scripts/webkitpy/common/system/filesystem.py (285778 => 285779)
--- trunk/Tools/Scripts/webkitpy/common/system/filesystem.py 2021-11-13 21:57:32 UTC (rev 285778)
+++ trunk/Tools/Scripts/webkitpy/common/system/filesystem.py 2021-11-13 21:58:50 UTC (rev 285779)
@@ -339,6 +339,3 @@
self.copytree(source, destination)
else:
self.copyfile(source, destination)
-
- def symlink(self, *args, **kwargs):
- os.symlink(*args, **kwargs)
Modified: trunk/Tools/Scripts/webkitpy/common/system/filesystem_mock.py (285778 => 285779)
--- trunk/Tools/Scripts/webkitpy/common/system/filesystem_mock.py 2021-11-13 21:57:32 UTC (rev 285778)
+++ trunk/Tools/Scripts/webkitpy/common/system/filesystem_mock.py 2021-11-13 21:58:50 UTC (rev 285779)
@@ -436,10 +436,7 @@
def copy_from_base_host(self, source, destination):
self.move(source, destination)
- def symlink(self, src, dst):
- self.move(src, dst)
-
class WritableBinaryFileObject(object):
def __init__(self, fs, path):
self.fs = fs
Modified: trunk/Tools/Scripts/webkitpy/port/darwin.py (285778 => 285779)
--- trunk/Tools/Scripts/webkitpy/port/darwin.py 2021-11-13 21:57:32 UTC (rev 285778)
+++ trunk/Tools/Scripts/webkitpy/port/darwin.py 2021-11-13 21:58:50 UTC (rev 285779)
@@ -82,9 +82,6 @@
def _path_to_webcore_library(self):
return self._build_path('WebCore.framework/Versions/A/WebCore')
- def path_to_daemons(self):
- return self._build_path('WebKit.framework/Daemons')
-
def show_results_html_file(self, results_filename):
# We don't use self._run_script() because we don't want to wait for the script
# to exit and we want the output to show up on stdout in case there are errors
Modified: trunk/Tools/Scripts/webkitpy/xcode/simulated_device.py (285778 => 285779)
--- trunk/Tools/Scripts/webkitpy/xcode/simulated_device.py 2021-11-13 21:57:32 UTC (rev 285778)
+++ trunk/Tools/Scripts/webkitpy/xcode/simulated_device.py 2021-11-13 21:58:50 UTC (rev 285779)
@@ -60,7 +60,6 @@
self.version = Version.from_string(runtime_dict['version'])
self.identifier = runtime_dict['identifier']
self.name = runtime_dict['name']
- self.root = runtime_dict.get('runtimeRoot')
AVAILABLE_RUNTIMES = []
AVAILABLE_DEVICES = []
@@ -124,7 +123,6 @@
host=host,
device_type=device_type,
build_version=runtime.build_version,
- runtime=runtime,
))
SimulatedDeviceManager.AVAILABLE_DEVICES.append(result)
return result
@@ -546,7 +544,7 @@
'watchOS': 'com.apple.carousel.sessionservice',
}
- def __init__(self, name, udid, host, device_type, build_version, runtime):
+ def __init__(self, name, udid, host, device_type, build_version):
assert device_type.software_version
self.name = name
@@ -553,7 +551,6 @@
self.udid = udid
self.device_type = device_type
self.build_version = build_version
- self.runtime = runtime
self._state = SimulatedDevice.DeviceState.SHUTTING_DOWN
self.executive = host.executive
Modified: trunk/Tools/Scripts/webkitpy/xcode/simulated_device_unittest.py (285778 => 285779)
--- trunk/Tools/Scripts/webkitpy/xcode/simulated_device_unittest.py 2021-11-13 21:57:32 UTC (rev 285778)
+++ trunk/Tools/Scripts/webkitpy/xcode/simulated_device_unittest.py 2021-11-13 21:58:50 UTC (rev 285779)
@@ -168,8 +168,7 @@
"availability" : "(available)",
"name" : "iOS 9.3",
"identifier" : "com.apple.CoreSimulator.SimRuntime.iOS-9-3",
- "version" : "9.3",
- "runtimeRoot" : "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 9.3.simruntime/Contents/Resources/RuntimeRoot"
+ "version" : "9.3"
},
{
"buildversion" : "15A8401",
@@ -176,8 +175,7 @@
"availability" : "(available)",
"name" : "iOS 11.0",
"identifier" : "com.apple.CoreSimulator.SimRuntime.iOS-11-0",
- "version" : "11.0.1",
- "runtimeRoot" : "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 11.0.1.simruntime/Contents/Resources/RuntimeRoot"
+ "version" : "11.0.1"
},
{
"buildversion" : "15J380",
@@ -184,8 +182,7 @@
"availability" : "(available)",
"name" : "tvOS 11.0",
"identifier" : "com.apple.CoreSimulator.SimRuntime.tvOS-11-0",
- "version" : "11.0",
- "runtimeRoot" : "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/tvOS 11.simruntime/Contents/Resources/RuntimeRoot"
+ "version" : "11.0"
},
{
"buildversion" : "15R372",
@@ -192,8 +189,7 @@
"availability" : "(available)",
"name" : "watchOS 4.0",
"identifier" : "com.apple.CoreSimulator.SimRuntime.watchOS-4-0",
- "version" : "4.0",
- "runtimeRoot" : "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/watchOS 4.simruntime/Contents/Resources/RuntimeRoot"
+ "version" : "4.0"
},
{
"buildversion" : "16A367",
@@ -200,8 +196,7 @@
"isAvailable" : "YES",
"name" : "iOS 12.0",
"identifier" : "com.apple.CoreSimulator.SimRuntime.iOS-12-0",
- "version" : "12.0",
- "runtimeRoot" : "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 12.simruntime/Contents/Resources/RuntimeRoot"
+ "version" : "12.0"
}
],
"devices" : {
