Bug 108937 - [Qt] Broken build for WebCore qt/tests since r141598
Summary: [Qt] Broken build for WebCore qt/tests since r141598
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Major
Assignee: Nobody
Depends on:
Reported: 2013-02-05 06:34 PST by Julien Brianceau
Modified: 2013-02-05 11:21 PST (History)
3 users (show)

See Also:

Build fix (1.12 KB, patch)
2013-02-05 06:40 PST, Julien Brianceau
jbriance: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Brianceau 2013-02-05 06:34:44 PST
WebCore qt/tests build is broken since r141598
Seen with our sh4 build slave: http://build.webkit.org/builders/Qt%20Linux%20SH4%20Release/builds/19293/steps/compile-webkit/logs/stdio
Comment 1 Julien Brianceau 2013-02-05 06:40:31 PST
Created attachment 186617 [details]
Build fix
Comment 2 Simon Hausmann 2013-02-05 07:26:30 PST
Comment on attachment 186617 [details]
Build fix

Did you want to set r? instead of r-?
Comment 3 Jocelyn Turcotte 2013-02-05 07:35:23 PST
I think that his Qt is old or something, QSignalSpy takes a const QObject* since October in qtbase from what I've seen.
Comment 4 Julien Brianceau 2013-02-05 07:36:59 PST
(In reply to comment #2)
> (From update of attachment 186617 [details])
> Did you want to set r? instead of r-?
No, the r- is my intention (thanks to Allan).

QSignalSpy prototype changed in Qt5 (in commit id 80ae1d2078d2fe137f8a88268c66440bf19bc1ba) and our build slave doesn't seem to be up to date..
I'm going to update our bot and see if it's better, I'll keep you informed.
Comment 5 Julien Brianceau 2013-02-05 11:21:08 PST
That's it. The qt5 build was too old in our build slave.
I performed a fast-forward to the qt5 5.0.0 release (git tag "v5.0.0") and now it's ok:

Thanks and sorry for the noise