WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 203056
Simulator testers are filling up
https://bugs.webkit.org/show_bug.cgi?id=203056
Summary
Simulator testers are filling up
Jonathan Bedard
Reported
2019-10-16 14:51:21 PDT
Some of our simulator testers are filling up their hardrives alarmingly quickly, probably due to our habit of recreating simulators constantly. We should do some cleanup every time we tear down automatically created simulators.
Attachments
Patch
(1.72 KB, patch)
2019-10-16 15:04 PDT
,
Jonathan Bedard
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Jonathan Bedard
Comment 1
2019-10-16 14:51:50 PDT
<
rdar://problem/56344905
>
Jonathan Bedard
Comment 2
2019-10-16 15:04:19 PDT
Created
attachment 381119
[details]
Patch
Aakash Jain
Comment 3
2019-10-30 11:40:59 PDT
Comment on
attachment 381119
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=381119&action=review
> Tools/Scripts/webkitpy/xcode/simulated_device.py:483 > + for directory in host.filesystem.glob('/tmp/com.apple.CoreSimulator.SimDevice.*'):
will this work fine for windows?
Jonathan Bedard
Comment 4
2019-10-30 11:44:43 PDT
Comment on
attachment 381119
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=381119&action=review
>> Tools/Scripts/webkitpy/xcode/simulated_device.py:483 >> + for directory in host.filesystem.glob('/tmp/com.apple.CoreSimulator.SimDevice.*'): > > will this work fine for windows?
No, but you can't initialize simulators if you aren't on a Mac, and if we don't have initialized simulators, we early return on line 470.
WebKit Commit Bot
Comment 5
2019-10-30 12:43:42 PDT
Comment on
attachment 381119
[details]
Patch Clearing flags on attachment: 381119 Committed
r251793
: <
https://trac.webkit.org/changeset/251793
>
WebKit Commit Bot
Comment 6
2019-10-30 12:43:44 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug