Summary: | Add script that filters the output of build-webkit to be more human-readable | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Matthew Delaney <mdelaney7> | ||||
Component: | Tools / Tests | Assignee: | Matthew Delaney <mdelaney7> | ||||
Status: | RESOLVED DUPLICATE | ||||||
Severity: | Normal | CC: | ddkilzer, eric, jamesr, joepeck, pknight | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Matthew Delaney
2011-09-02 11:47:38 PDT
I agree! I used to even have one of such scripts... Do you know why the logs are truncated on the mac build bots right now? Doing 'build-webkit | grep -v setenv' as kilzer suggests would be a super fast for the short term. Any chance we could throw that in there on the bots? Nevermind, I've got a script that's awesome that's been floating around. I'll throw a patch up for it soon here. Created attachment 106193 [details]
Patch
Duplicate? The patch is an updated version of the patch on: <http://webkit.org/b/44081> build-webkit output could be easier to read in most situations Comment on attachment 106193 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=106193&action=review > Tools/Scripts/filter-build-webkit:3 > +# Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved. Really? You've been working on this script a long time! > Tools/Scripts/filter-build-webkit:196 > +sub setLogileOption($$) Logile? Comment on attachment 106193 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=106193&action=review > Tools/Scripts/filter-build-webkit:147 > + elsif ($line =~ /^(PhaseScriptExecution|CompileC|Ld|PBXCp|CpResource|CopyPNGFile|CopyTiffFile|CpHeader|Processing|ProcessInfoPlistFile|ProcessPCH|ProcessPCH\+\+|Touch|Libtool|CopyStringsFile|Mig|CreateUniversalBinary) ("[^"]+"|\S+)/) { I'd suggest "Analyze" here as well for clang's static analyzer. For simplicity, I'm duping this against bug 44081 as Joe suggests since it has the old version patch of the script there with other ideas for it and how to incorporate it into the actual build-webkit script. *** This bug has been marked as a duplicate of bug 44081 *** Comment on attachment 106193 [details] Patch Cleared review? from attachment 106193 [details] so that this bug does not appear in http://webkit.org/pending-review. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again). |