| Summary: | [Qt] When building QtWebKit with the -ltcg configure option then it will fail to build | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | andy.shaw | ||||||
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | benjamin, pierre.rossi | ||||||
| Priority: | P3 | Keywords: | Qt | ||||||
| Version: | 528+ (Nightly build) | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Attachments: |
|
||||||||
Created attachment 120167 [details]
Patch against trunk
Comment on attachment 120167 [details]
Patch against trunk
rs=me but the patch will have to be applied manually because it's missing the Tools/ directory prefix.
(In reply to comment #2) > (From update of attachment 120167 [details]) > rs=me but the patch will have to be applied manually because it's missing the Tools/ directory prefix. ...and things had moved back to Source/WebCore in the meantime ;) Landed in r104758 Thanks! Will this be included in Qt 4.8.1 as it is a build fix? If so then I will not push it to the internal group of patches we have for Qt Commercial 4.8.1. Hi Andy, nice to see you here :) |
Created attachment 120165 [details] Fix for the build issue The reason it fails is because -GL is still specified for the compile options which is still there despite lines in WebCore.pri removing this from the flags. The reason being because it did not remove it from the right set of flags. Patch attached fixes this so it is removed from the right set of flags.