Bug 222066 - Skip platform specific Python tests
Summary: Skip platform specific Python tests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Don Olmstead
URL:
Keywords: InRadar
Depends on:
Blocks: 221815
  Show dependency tree / graph
 
Reported: 2021-02-17 13:52 PST by Don Olmstead
Modified: 2021-02-18 12:16 PST (History)
4 users (show)

See Also:

Attachments
Patch (2.45 KB, patch)
2021-02-17 14:01 PST, Don Olmstead 		no flags Details | Formatted Diff | Diff
Patch (2.45 KB, patch)
2021-02-17 14:24 PST, Don Olmstead 		no flags Details | Formatted Diff | Diff
Patch (2.61 KB, patch)
2021-02-17 15:39 PST, Don Olmstead 		no flags Details | Formatted Diff | Diff
Patch (2.67 KB, patch)
2021-02-17 15:49 PST, Don Olmstead 		no flags Details | Formatted Diff | Diff
Patch (2.62 KB, patch)
2021-02-17 15:50 PST, Don Olmstead 		no flags Details | Formatted Diff | Diff
Patch (2.62 KB, patch)
2021-02-17 15:51 PST, Don Olmstead 		no flags Details | Formatted Diff | Diff
Show Obsolete (5) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Don Olmstead 2021-02-17 13:52:01 PST 
There's no reason to run Mac unit tests on Linux or Windows. No reason to run Linux unit tests on Mac or Windows. No reason to run Windows unit tests on Mac or Linux.
Comment 1 Don Olmstead 2021-02-17 14:01:09 PST Comment hidden (obsolete)
Comment 2 Don Olmstead 2021-02-17 14:24:02 PST Comment hidden (obsolete)
Comment 3 Don Olmstead 2021-02-17 15:39:20 PST Comment hidden (obsolete)
Comment 4 Don Olmstead 2021-02-17 15:49:49 PST Comment hidden (obsolete)
Comment 5 Don Olmstead 2021-02-17 15:50:52 PST Comment hidden (obsolete)
Comment 6 Don Olmstead 2021-02-17 15:51:45 PST 
Created attachment 420741 [details]
Patch
Comment 7 Jonathan Bedard 2021-02-17 16:29:36 PST 
Comment on attachment 420741 [details]
Patch

Looks good, holding off on r+ing until tomorrow when the Igalia folks have had a chance to give their 2 cents about Linux
Comment 8 Don Olmstead 2021-02-18 08:47:23 PST 
Comment on attachment 420741 [details]
Patch

cq+ing because Lauro approved in Slack
Comment 9 EWS 2021-02-18 12:15:55 PST 
Committed r273092: <https://commits.webkit.org/r273092>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 420741 [details].
Comment 10 Radar WebKit Bug Importer 2021-02-18 12:16:14 PST 
<rdar://problem/74489266>