Bug 196657

Summary: [ews-build] Make PrintConfiguration platform aware
Product: WebKit Reporter: Aakash Jain <aakash_jain>
Component: Tools / TestsAssignee: Aakash Jain <aakash_jain>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, aperez, ap, clopez, commit-queue, ews-watchlist, lforschler, mcatanzaro, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=199234
Attachments:
Description Flags
Patch
none
Patch none

Description Aakash Jain 2019-04-05 14:42:29 PDT
PrintConfiguration step currently fails on windows as it runs macOS specific commands (e.g.: xcodebuild). We should add an OS check in there.
Comment 1 Aakash Jain 2019-04-05 14:46:14 PDT
Created attachment 366844 [details]
Patch
Comment 2 EWS Watchlist 2019-04-05 14:50:58 PDT
Attachment 366844 [details] did not pass style-queue:


ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:912:  [PrintConfiguration.run] Use of super on an old style class  [pylint/E1002] [5]
ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:913:  [PrintConfiguration.run] Instance of 'PrintConfiguration' has no 'getProperty' member  [pylint/E1101] [5]
Total errors found: 2 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Aakash Jain 2019-07-02 04:57:57 PDT
Created attachment 373314 [details]
Patch
Comment 4 WebKit Commit Bot 2019-07-02 07:43:03 PDT
Comment on attachment 373314 [details]
Patch

Clearing flags on attachment: 373314

Committed r247049: <https://trac.webkit.org/changeset/247049>
Comment 5 WebKit Commit Bot 2019-07-02 07:43:04 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Radar WebKit Bug Importer 2019-07-02 07:44:19 PDT
<rdar://problem/52522637>