[webkit-qt] QtWebKit 2.1 branch week 8 ftbfs

[Caio Marcelo de Oliveira Filho]

Hello,

> Yes, I have Qt Mobility master branch (upcoming 1.2).
>
> with the same setup, week7 tag builds fine.
> I work around the issue with the following patch:
>
> --- a/WebCore/WebCore.pro
> +++ b/WebCore/WebCore.pro
> @@ -2572,13 +2572,11 @@ contains(DEFINES, ENABLE_VIDEO=1) {
>         bindings/js/JSAudioConstructor.cpp
>
>         !lessThan(QT_MINOR_VERSION, 6):contains(MOBILITY_CONFIG, multimedia) {
> -            HEADERS += \
> -                platform/graphics/qt/MediaPlayerPrivateQt.h \
> -                $$PWD/../WebKit/qt/WebCoreSupport/FullScreenVideoQt.h
> +            HEADERS += \
> +                platform/graphics/qt/MediaPlayerPrivateQt.h
>
>             SOURCES += \
> -                platform/graphics/qt/MediaPlayerPrivateQt.cpp \
> -                $$PWD/../WebKit/qt/WebCoreSupport/FullScreenVideoQt.cpp
> +                platform/graphics/qt/MediaPlayerPrivateQt.cpp
>
>             CONFIG *= mobility
>             MOBILITY += multimedia

I looked at the file you mentioned in the diff (WebCore/WebCore.pro)
in the gitorious web visualization of the tree for 2.1 branch

    http://gitorious.org/+qtwebkit-developers/webkit/qtwebkit/blobs/qtwebkit-2.1/WebCore/WebCore.pro

and they do not mention "FullScreenVideoQt". As Alexis pointed out,
this shouldn't be on 2.1 branch.

Could you double check if its really the 2.1 branch you're compiling?
Maybe you've applied some patch on top of your 2.1 tree. The latest
commit on it (in the week09 tag) is
01f01edc8dde851f60a1067190e9a819c9bb024a.


Cheers,

-- 
Caio Marcelo de Oliveira Filho
OpenBossa - INdT