| Summary: | [ews-build] Add build step to UnApply patch | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||||
| Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | aakash_jain, ap, dbates, ddkilzer, ews-watchlist, lforschler, webkit-bug-importer | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | Other | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=187358 | ||||||||
| Attachments: |
|
||||||||
|
Description
Aakash Jain
2018-07-05 12:41:50 PDT
Created attachment 344360 [details] Proposed patch Tested in http://ews-build.webkit-uat.org/#/builders/9/builds/22 Attachment 344360 [details] did not pass style-queue:
ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:72: [UnApplyPatchIfRequired.__init__] Use of super on an old style class [pylint/E1002] [5]
Total errors found: 1 in 2 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 344360 [details] Proposed patch Attachment 344360 [details] did not pass win-ews (win): Output: https://webkit-queues.webkit.org/results/8450316 New failing tests: http/tests/security/local-video-source-from-remote.html Created attachment 344385 [details]
Archive of layout-test-results from ews204 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews204 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Committed r233708: <https://trac.webkit.org/changeset/233708> Comment on attachment 344360 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=344360&action=review > Tools/BuildSlaveSupport/ews-build/steps.py:73 > + super(UnApplyPatchIfRequired, self).__init__(alwaysUseLatest=True, **kwargs) From my understanding setting alwaysUseLatest to true both cleans the working directory and pulls down new source changes. This represents a behavior change from the current EWS. The current EWS actually unapplies the patch. It does this implicitly by cleaning the working directory. That is, it does not both clean the working directory and pull down new source changes. |