The WinCairo build fails because the configuration names has changed in the VS2010 project, and needs to be updated in the scripts.
Created attachment 203851 [details] Patch
Comment on attachment 203851 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=203851&action=review This patch looks straightforward to me. r=me > Tools/ChangeLog:3 > + [WinCairo] Build failure. Do we know when the build began failing? From briefly looking through SVN history it looks like the WinCairo broken during the transition from VS2005 to VS2010. Maybe a more appropriate bug title would be "Fix build-webkit to build WinCairo under VS2010"?
(In reply to comment #2) > > Tools/ChangeLog:3 > > + [WinCairo] Build failure. > > Do we know when the build began failing? From briefly looking through SVN history it looks like the WinCairo broken during the transition from VS2005 to VS2010. Maybe a more appropriate bug title would be "Fix build-webkit to build WinCairo under VS2010"? "WinCairo broken during" => "WinCairo build has been broken since"
(In reply to comment #2) > (From update of attachment 203851 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=203851&action=review > > This patch looks straightforward to me. > > r=me > > > Tools/ChangeLog:3 > > + [WinCairo] Build failure. > > Do we know when the build began failing? From briefly looking through SVN history it looks like the WinCairo broken during the transition from VS2005 to VS2010. Maybe a more appropriate bug title would be "Fix build-webkit to build WinCairo under VS2010"? I'm not sure exactly when the WinCairo build started to fail.
Comment on attachment 203851 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=203851&action=review >>> Tools/ChangeLog:3 >>> + [WinCairo] Build failure. >> >> Do we know when the build began failing? From briefly looking through SVN history it looks like the WinCairo broken during the transition from VS2005 to VS2010. Maybe a more appropriate bug title would be "Fix build-webkit to build WinCairo under VS2010"? > > I'm not sure exactly when the WinCairo build started to fail. OK. Please update the bug title in the ChangeLog entry before landing this patch.
Created attachment 203910 [details] Patch
(In reply to comment #5) > > OK. Please update the bug title in the ChangeLog entry before landing this patch. Updated the title in the ChangeLog.
Comment on attachment 203910 [details] Patch Thanks peavo for updating the patch. r=me
(In reply to comment #8) > (From update of attachment 203910 [details]) > Thanks peavo for updating the patch. > r=me Thanks for the review :)
Comment on attachment 203910 [details] Patch Clearing flags on attachment: 203910 Committed r151271: <http://trac.webkit.org/changeset/151271>
All reviewed patches have been landed. Closing bug.