Bug 29677 - Guard QX11Info include with Q_WS_X11
Summary: Guard QX11Info include with Q_WS_X11
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 27065
  Show dependency treegraph
 
Reported: 2009-09-23 05:19 PDT by Norbert Leser
Modified: 2009-09-23 09:23 PDT (History)
7 users (show)

See Also:


Attachments
QX11Info patch (1.46 KB, patch)
2009-09-23 05:19 PDT, Norbert Leser
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Norbert Leser 2009-09-23 05:19:56 PDT
Created attachment 39990 [details]
QX11Info patch

Need to guard QX11Info include with Q_WS_X11. That class may not be available (in QT 4.5 for Symbian, for instance). Completes fixes in r48627 and r48604.
Comment 1 Kenneth Rohde Christiansen 2009-09-23 05:50:30 PDT
Norbert, you don't need reviews for build fixes like this one :-) Feel free to commit! Just write "Unreviewed Qt build fix" in the ChangeLog
Comment 2 Tor Arne Vestbø 2009-09-23 05:55:51 PDT
Comment on attachment 39990 [details]
QX11Info patch

Kennth is right, but r=me
Comment 3 Norbert Leser 2009-09-23 06:00:43 PDT
(In reply to comment #2)
> (From update of attachment 39990 [details])
> Kennth is right, but r=me

I don't have commit rights - thus, someone else please pick it up.

The fix seems trivial, and in this case, I assume it is - still, these sort of changes can easily break the build on certain platforms, under certain conditions. I would caution to treat these as pass-through / non-review fixes.
Comment 4 WebKit Commit Bot 2009-09-23 06:08:55 PDT
Comment on attachment 39990 [details]
QX11Info patch

Rejecting patch 39990 from commit-queue.

kenneth.christiansen@openbossa.org does not have committer permissions according to http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/modules/committers.py.
Comment 5 Kenneth Rohde Christiansen 2009-09-23 06:15:03 PDT
kenneth@webkit.org is an alias for kenneth.christiansen@openbossa.org

Committer("Kenneth Rohde Christiansen", "kenneth@webkit.org"),

Should I add both or ?
Comment 6 Kenneth Rohde Christiansen 2009-09-23 06:21:20 PDT
Comment on attachment 39990 [details]
QX11Info patch

Let's try again! I changed my bugzilla e-mail.
Comment 7 WebKit Commit Bot 2009-09-23 09:23:54 PDT
Comment on attachment 39990 [details]
QX11Info patch

Clearing flags on attachment: 39990

Committed r48676: <http://trac.webkit.org/changeset/48676>
Comment 8 WebKit Commit Bot 2009-09-23 09:23:59 PDT
All reviewed patches have been landed.  Closing bug.