[Webkit-unassigned] [Bug 116522] New: [Qt][MAC] r150277 broke Qt Mountain Lion Release build

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=116522

           Summary: [Qt][MAC] r150277 broke Qt Mountain Lion Release build
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: Qt
          Severity: Normal
          Priority: P2
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: zarvai at inf.u-szeged.hu
                CC: andersca at apple.com, sam at webkit.org, abecsi at webkit.org,
                    jocelyn.turcotte at digia.com, kadam at inf.u-szeged.hu
            Blocks: 116335


Build fails after new header added:

#if defined(__APPLE__) 
#include <WebKit2/WKBaseMac.h> 
#endif 

In file included from /Users/admin/work/WebKit-BuildSlave/qt-mountainlion-release/build/Source/WebKit2/Shared/API/c/WKGeometry.cpp:27:
In file included from /Users/admin/work/WebKit-BuildSlave/qt-mountainlion-release/build/Source/WebKit2/Shared/API/c/WKGeometry.h:29:
In file included from ../include/WebKit2/WKBase.h:1:
/Users/admin/work/WebKit-BuildSlave/qt-mountainlion-release/build/Source/WebKit2/Shared/API/c/WKBase.h:45:10: fatal error: 'WebKit2/WKBaseMac.h' file not found
#include <WebKit2/WKBaseMac.h>

Probably, this "defined(__APPLE__)" should be different.
Which guard would be better? "PLATFORM(MAC)" or "defined(__APPLE__) && !PLATFORM(QT)"

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.