Summary: | [Qt] Webkit for symbian building fails | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | t.zander | ||||||
Component: | WebKit Qt | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | CLOSED FIXED | ||||||||
Severity: | Normal | CC: | hausmann, webkit.review.bot | ||||||
Priority: | P2 | Keywords: | Qt | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | S60 3rd edition | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 35784 | ||||||||
Attachments: |
|
Attachment 51888 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
Total errors found: 0 in 0 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 51888 [details]
patch
Looks okay to me in principle, but can you explain why you're excluding these two statements?
Please also add a link to the bugzilla entry in the ChangeLog?
Before the only symbian build system seeing those lines were the Windows (aka MMP) based ones. Now the linux system sees them too. And those lines are not valid there so they have to be excluded specifically. Do you want me to re-submit it re-indented? (In reply to comment #3) > Before the only symbian build system seeing those lines were the Windows (aka > MMP) based ones. Now the linux system sees them too. And those lines are not > valid there so they have to be excluded specifically. > > Do you want me to re-submit it re-indented? The indentation is fine, but it would be good to have the explanation in the ChangeLog or to have a link from the ChangeLog to this bugzilla entry, so that it's possible to find out what happened when looking at the .pro file in three months :) Created attachment 51897 [details]
updated patch
Comment on attachment 51897 [details]
updated patch
Ok! For the purpose of getting this into the release soon, I'll also land it manually.
Committed r56715: <http://trac.webkit.org/changeset/56715> |
Created attachment 51888 [details] patch to make webkit compile on a linux box using gcce and cross compiling to arm for symbian I found that the attached change was required.