Bug 142034

Summary: [Win] Make build logs more legible by reducing noise
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebCore Misc.Assignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, bfulgham, darin, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
Patch ap: review+

Description Brent Fulgham 2015-02-25 17:52:58 PST
Our Windows build logs produce a tremendous amount of boilerplate output and detailed build state that makes it very difficult to find and understand why the build has failed.

We can improve things as follows:

1. Update our batch file (Windows CMD files) to use the '@' character as the first character on each line. This will prevent these lines from being echoed into our build logs.
2. Pass the '/Q' flag to our invocations of CMD so that they run silently. This will reduce another set of output that we generally do not care about.
3. Pipe the output of XCOPY to "NUL". This will hide the file copy state, but we have not generally found these messages to be helpful when debugging build errors, even when they have been caused by files not being copied.
Comment 1 Brent Fulgham 2015-02-25 22:54:42 PST
Created attachment 247410 [details]
Patch
Comment 2 Brent Fulgham 2015-02-25 23:04:47 PST
The patch seems to be failing on the Windows EWS bot, I believe due to EOL differences. 

Unfortunately, I can't tell from the log (which is ironic, since the point of this patch is to fix this problem).
Comment 3 Alexey Proskuryakov 2015-02-25 23:35:09 PST
Comment on attachment 247410 [details]
Patch

rs=me
Comment 4 Brent Fulgham 2015-02-26 08:51:25 PST
Committed r180676: <http://trac.webkit.org/changeset/180676>
Comment 5 Brent Fulgham 2015-02-26 10:20:29 PST
I missed an important set of changes to {Pre|Post}{Build|Link} steps. This was committed in r180680 <https://trac.webkit.org/changeset/180680>.
Comment 6 Radar WebKit Bug Importer 2015-02-27 10:21:02 PST
<rdar://problem/19985941>