Bug 158510

Summary: Don't try to cleanup non-existing ios Simulators
Product: WebKit Reporter: Aakash Jain <aakash_jain>
Component: Tools / TestsAssignee: Aakash Jain <aakash_jain>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, ap, commit-queue, glenn, lforschler
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Proposed patch
none
Proposed patch none

Aakash Jain
Reported 2016-06-07 18:47:54 PDT
When for some reason run-webkit-tests fails before creating the simulators, the cleanup still tries to cleanup the simulators. The “lsregister” command fails with the error message: “error -10814 unregistering /tmp/WebKitTestingSimulators/Simulator0.app” This just creates unnecessary logs and failed attempts. We should not try to cleanup non-existing simulators.
Attachments
Proposed patch (1.25 KB, patch)
2016-06-07 19:01 PDT, Aakash Jain 		no flags
DetailsFormatted DiffDiff
Proposed patch (1.24 KB, patch)
2016-06-07 19:10 PDT, Aakash Jain 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2016-06-07 19:01:59 PDT
Created attachment 280760 [details] Proposed patch
Aakash Jain
Comment 2 2016-06-07 19:10:24 PDT
Created attachment 280761 [details] Proposed patch
WebKit Commit Bot
Comment 3 2016-06-08 11:01:39 PDT
Comment on attachment 280761 [details] Proposed patch Clearing flags on attachment: 280761 Committed r201812: <http://trac.webkit.org/changeset/201812>
WebKit Commit Bot
Comment 4 2016-06-08 11:01:43 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.