Bug 101345

Summary: [EFL] Switch the efl-linux-slave-2 to Wk2 Release
Product: WebKit Reporter: Dominik Röttsches (drott) <d-r>
Component: WebKit EFLAssignee: Dominik Röttsches (drott) <d-r>
Status: RESOLVED FIXED    
Severity: Normal CC: bweinstein, dpranke, lforschler, lucas.de.marchi, naginenis, rwlbuis, webkit.review.bot, wsiegrist
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 101444    
Attachments:
Description Flags
Patch
none
Patch v2, fixed build-dir. none

Dominik Röttsches (drott)
Reported 2012-11-06 06:52:06 PST
We'd like to focus on WebKit2 robustness and concentrate our gardening on that, hence moving our Wk1 Debug bot to Wk2 release.
Attachments
Patch (2.62 KB, patch)
2012-11-06 06:59 PST, Dominik Röttsches (drott) 		no flags
DetailsFormatted DiffDiff
Patch v2, fixed build-dir. (2.63 KB, patch)
2012-11-07 00:59 PST, Dominik Röttsches (drott) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dominik Röttsches (drott)
Comment 1 2012-11-06 06:59:35 PST
Created attachment 172577 [details] Patch
Dominik Röttsches (drott)
Comment 2 2012-11-06 07:01:01 PST
Lucas, could you informally review this? Thanks.
Dominik Röttsches (drott)
Comment 3 2012-11-07 00:59:30 PST
Created attachment 172730 [details] Patch v2, fixed build-dir.
Dominik Röttsches (drott)
Comment 4 2012-11-07 01:02:38 PST
Gyuyoung, I changed the builddir still, could you take another quick look and r+? Then I'd land this and watch whether the bot picks it up correctly. Thanks.
Gyuyoung Kim
Comment 5 2012-11-07 04:00:54 PST
Comment on attachment 172730 [details] Patch v2, fixed build-dir. Please look at buildbots after landing this patch.
WebKit Review Bot
Comment 6 2012-11-07 04:28:58 PST
Comment on attachment 172730 [details] Patch v2, fixed build-dir. Clearing flags on attachment: 172730 Committed r133735: <http://trac.webkit.org/changeset/133735>
WebKit Review Bot
Comment 7 2012-11-07 04:29:02 PST
All reviewed patches have been landed. Closing bug.
Dominik Röttsches (drott)
Comment 8 2012-11-07 05:14:49 PST
William, it looks like the master did not pick up the change, could you restart it?
Lucas Forschler
Comment 9 2012-11-08 10:11:13 PST
I have verified that the code on the master has been updated. It's currently at r133854. This patch landed in 133735. I also restarted the master. I'm going to take a look at the config.json and see if something else was missed.
Dominik Röttsches (drott)
Comment 10 2012-11-15 07:35:39 PST
Thanks for taking a look - whatever convinced it, in the end our bot now switched to the new config.
Note You need to log in before you can comment on or make changes to this bug.