After switching MSVC build to Qt5-r37 (749b83f337ec82eff305ed8ac85473ad984c84fa) a build error occurred: WebKit is now configured for building. Just run 'make'. Last built revision was unknown, now at revision 130570. Full incremental build needed. Calling 'make -j8 incremental' in C:\WebKitBuildSlave\szeged-windows-1\qt-windows-32bit-release\build\WebKitBuild\Release 'make' is not recognized as an internal or external command, operable program or batch file. Wrong make command is used. It should be nmake instead of make. The problem caused by qmake.conf and webkitdirs.pm. Qt5.0.0-r37-angle\mkspecs\default\qmake.conf contains: QMAKESPEC_ORIGINAL = C:/WebKitBuildSlave/Qt5.0.0-r37-source/qt5/qtbase/mkspecs/win32-msvc2010 include($$QMAKESPEC_ORIGINAL/qmake.conf) WebKit\Tools\Scripts\webkitdirs.pm contains a function called retrieveQMakespecVar. This function parses qmake.conf. It can follow includes, but it can not resolve variables. The workaround for this issue is changing qmake.conf to the old value: QMAKESPEC_ORIGINAL = C:/WebKitBuildSlave/Qt5.0.0beta1-source/qt5/qtbase/mkspecs/win32-msvc2010 include(../win32-msvc2010/qmake.conf)
Created attachment 173000 [details] Patch
Created attachment 173003 [details] Patch
It is fixed in Qt - https://bugreports.qt-project.org/browse/QTBUG-27593 So I think we won't need this fix after we use fixed Qt.
(In reply to comment #3) > It is fixed in Qt - https://bugreports.qt-project.org/browse/QTBUG-27593 > So I think we won't need this fix after we use fixed Qt. This is exactly what this patch builds on :). No more default/ but instead the mkspec is coded "into" qmake. We now query it out of qmake with -query QMAKE_XMKSPEC.
Comment on attachment 173003 [details] Patch Rejecting attachment 173003 [details] from commit-queue. New failing tests: fast/repaint/selection-after-delete.html Full output: http://queues.webkit.org/results/14755937
Committed r134024: <http://trac.webkit.org/changeset/134024>
It broke our Windows bot: Calling 'make -j8 incremental' in C:\WebKitBuildSlave\szeged-windows-1\qt-windows-32bit-release\build\WebKitBuild\Release 'make' is not recognized as an internal or external command, operable program or batch file. Zoltán, could you check it? Maybe the workaround and this patch conflicts.
Submitted http://trac.webkit.org/changeset/134032 to accomodate older Qt versions that don't support QMAKE_XSPEC yet but instead do install a "default" folder.