Bug 108422

Summary: REGRESSION(r141349): "yet another windows ews fix needed" (Requested by lforschler on #webkit).
Product: WebKit Reporter: WebKit Review Bot <webkit.review.bot>
Component: New BugsAssignee: WebKit Review Bot <webkit.review.bot>
Status: NEW ---    
Severity: Normal CC: abarth, jonlee, lforschler, roger_fong, slewis, thorton
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 107968    
Attachments:
Description Flags
ROLLOUT of r141349
none
patch thorton: review+

Description WebKit Review Bot 2013-01-30 21:00:43 PST
http://trac.webkit.org/changeset/141349 broke the build:
"yet another windows ews fix needed" (Requested by lforschler on #webkit).

This is an automatic bug report generated by the sheriff-bot. If this bug
report was created because of a flaky test, please file a bug for the flaky
test (if we don't already have one on file) and dup this bug against that bug
so that we can track how often these flaky tests case pain.

"Only you can prevent forest fires." -- Smokey the Bear
Comment 1 WebKit Review Bot 2013-01-30 21:01:12 PST
Created attachment 185671 [details]
ROLLOUT of r141349

Any committer can land this patch automatically by marking it commit-queue+.  The commit-queue will build and test the patch before landing to ensure that the rollout will be successful.  This process takes approximately 15 minutes.

If you would like to land the rollout faster, you can use the following command:

  webkit-patch land-attachment ATTACHMENT_ID

where ATTACHMENT_ID is the ID of this attachment.
Comment 2 Lucas Forschler 2013-01-30 21:02:45 PST
build-dumprendertree is failing...  
Looks like we need to get that working before these will ever work.  DRT does build on our build slaves ok, so we need to find out what is missing/needed on the EWS bots.

Failed to run "['/home/buildbot/WebKit/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=APPLE-EWS-1', 'build-and-test', '--no-clean', '--no-update', '--test', '--non-interactive', '--port=win']" exit_code: 2 cwd: /home/buildbot/WebKit

Running run-webkit-tests
Failed to run "['Tools/Scripts/run-webkit-tests', '--no-new-test-results', '--no-show-results', '--exit-after-n-failures=30', '--quiet']" exit_code: 1

Running build-dumprendertree
Compiling DumpRenderTree failed!
Failed to execute Tools/Scripts/old-run-webkit-tests at Tools/Scripts/run-webkit-tests line 106.

Failed to run "['Tools/Scripts/run-webkit-tests', '--no-new-test-results', '--no-show-results', '--exit-after-n-failures=30', '--quiet']" exit_code: 1
Running build-dumprendertree
Compiling DumpRenderTree failed!
Failed to execute Tools/Scripts/old-run-webkit-tests at Tools/Scripts/run-webkit-tests line 106.
Comment 3 Roger Fong 2013-01-31 18:28:41 PST
Created attachment 185911 [details]
patch

Fix to make Win EWS bots go!
Again!
Comment 4 Roger Fong 2013-01-31 18:30:07 PST
I'm about to submit the patch anyways so I'll cq- the rollout
Comment 5 Roger Fong 2013-01-31 18:37:03 PST
Just kidding. The bots aren't in a good state for EWS right now. I'll need to clean that up first. CQ+ing the rollout
Comment 6 WebKit Review Bot 2013-01-31 18:46:08 PST
Comment on attachment 185671 [details]
ROLLOUT of r141349

Clearing flags on attachment: 185671

Committed r141522: <http://trac.webkit.org/changeset/141522>
Comment 7 Roger Fong 2013-02-01 11:17:49 PST
Can someone review this? Thanks
Comment 8 Lucas Forschler 2013-02-01 11:26:15 PST
Comment on attachment 185911 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=185911&action=review

> Scripts/webkitpy/tool/steps/runtests.py:93
> +            print self._options.build_style;

Is this in here for debugging?  You probably want to remove it if so.
Comment 9 Roger Fong 2013-02-01 11:32:59 PST
(In reply to comment #8)
> (From update of attachment 185911 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=185911&action=review
> 
> > Scripts/webkitpy/tool/steps/runtests.py:93
> > +            print self._options.build_style;
> 
> Is this in here for debugging?  You probably want to remove it if so.

Oh yeah, that won't be there.
Comment 10 Lucas Forschler 2013-02-01 11:48:45 PST
looks good to me.  I don't have reviewer privelidges.  But, you can land this manually with the print statement removed.
Comment 11 Tim Horton 2013-02-01 12:48:18 PST
Comment on attachment 185911 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=185911&action=review

> ChangeLog:8
> +        Disabling --quiet option on Windows for now because that fails when we try to use /dev/null.

You should fix this on the other end, but it's fine for now.
Comment 12 Roger Fong 2013-02-01 12:51:44 PST
http://trac.webkit.org/changeset/141628
Hoepfully that fixes all the issues