RESOLVED FIXED 195851
[ews-build] Don't run clean builds on EWS builders 
https://bugs.webkit.org/show_bug.cgi?id=195851
Summary [ews-build] Don't run clean builds on EWS builders
Aakash Jain
Reported 2019-03-16 08:48:25 PDT
Builders on ews-build don't need to run clean builds. Incremental builds are fine for EWS. clean builds take a lot longer without much value addition for EWS. The current EWS also doesn't run clean builds, ews-build should do the same.
Attachments
Patch (1.24 KB, patch)
2019-03-16 08:49 PDT, Aakash Jain 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2019-03-16 08:49:16 PDT
Created attachment 364932 [details] Patch
Geoffrey Garen
Comment 2 2019-03-16 13:48:41 PDT
Comment on attachment 364932 [details] Patch r=me
WebKit Commit Bot
Comment 3 2019-03-16 20:50:52 PDT
Comment on attachment 364932 [details] Patch Clearing flags on attachment: 364932 Committed r243050: <https://trac.webkit.org/changeset/243050>
WebKit Commit Bot
Comment 4 2019-03-16 20:50:53 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 5 2019-03-16 20:51:16 PDT
<rdar://problem/48958641>
Jonathan Bedard
Comment 6 2019-03-18 09:01:50 PDT
Question about this change: What happens if we need a clean build for some reason? Is there a mechanism to, for example, force a clean build if the build fails?
Jonathan Bedard
Comment 7 2019-03-18 11:08:24 PDT
(In reply to Jonathan Bedard from comment #6) > Question about this change: What happens if we need a clean build for some > reason? Is there a mechanism to, for example, force a clean build if the > build fails? Discussed this on IRC with Aakash. Turns out, the code which cleans up after a patch fails to build removes the build directory. So we shouldn't need to do anything special to handle this case.
Note You need to log in before you can comment on or make changes to this bug.