WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
47414
[Qt] Get the symbian/linux-armcc compile working properly
https://bugs.webkit.org/show_bug.cgi?id=47414
Summary
[Qt] Get the symbian/linux-armcc compile working properly
Keith Kyzivat
Reported
2010-10-08 06:43:32 PDT
Currently, the symbian/linux-armcc compile is not working properly. Some changes in Qt's symbian/linux-armcc mkspec and webkit Qt project changes are needed in the 'symbian' sections to correct this functionality. 1.) $$QMAKE_MOC is not defined. This is fixed by updates in the qt 4.7 mkspec, however that has not been checked in. The required changes have not been checked into the Qt 4.7 branch yet. I'll attach a patch with the required changes. 2.) Symbian configurations were not taking into account that deployment will be to OUTPUT_DIR. 3.) "TARGET.EPOCSTACKSIZE = 0x14000 // 80 kB" in WebKit.pri symbian configuration section is using the wrong kind of comments. Should be using qmake '#' comments. Similarly, "EPOCHEAPSIZE 0x40000 0x2000000 // Min 256kB, Max 32MB" needed to be changed for the same reason. 4.) "linux-* {" section dealing with shared library rpaths is incorrect -- symbian/linux-armcc doesn't understand them, and was mistakenly selected based on the 'linux-' in it's name. 5.) DumpRenderTree is pulling in fontconfig on symbian/linux-armcc even though it doesn't exist. I'll have a patch to add to this shortly that addresses these problems.
Attachments
Qt 4.7 symbian/linux-armcc mkspec fix
(1.73 KB, patch)
2010-10-08 07:03 PDT
,
Keith Kyzivat
no flags
Details
Formatted Diff
Diff
Patch
(12.34 KB, patch)
2010-10-08 07:29 PDT
,
Keith Kyzivat
eric
: review+
commit-queue
: commit-queue-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Keith Kyzivat
Comment 1
2010-10-08 07:03:32 PDT
Created
attachment 70245
[details]
Qt 4.7 symbian/linux-armcc mkspec fix This attachment patches Qt 4.7 with updates to the symbian/linux-armcc mkspec to properly handle expansion of QMAKE_MOC QMAKE_UIC and QMAKE_IDC
Keith Kyzivat
Comment 2
2010-10-08 07:29:18 PDT
Created
attachment 70249
[details]
Patch
Keith Kyzivat
Comment 3
2010-10-11 12:01:54 PDT
FYI qmake does not need to be rebuilt after applying the patch from
comment 1
, as it is just changes to the symbian/linux-armcc mkspec, so just apply and then continue on to apply and build with the webkit patch I have provided.
Kristian Amlie
Comment 4
2010-10-18 04:52:09 PDT
With my limited knowledge of the WebKit build structure I can't do final review, but the changes to file locations look good. For the TARGET.EPOCHEAPSIZE, note that anything added to MMP_RULES has no effect on the symbian/linux-armcc mkspec. Therefore the change you make to comments there is unnecessary. Although I don't see why they do, if they want to define it differently like that, you'll need a separate scope, like this: symbian-abld|symbian-sbsv2 { TARGET.EPOCHEAPSIZE = heapSizeRule = \ "$${LITERAL_HASH}ifdef WINSCW" \ "EPOCHEAPSIZE 0x40000 0x2000000" \ "$${LITERAL_HASH}else" \ "EPOCHEAPSIZE 0x40000 0x6000000" \ "$${LITERAL_HASH}endif" MMP_RULES += heapSizeRule } else { TARGET.EPOCHEAPSIZE = 0x40000 0x2000000 } Also, all the changes that change linux-* into linux-*:!symbian should be regarded as temporary, since the final mkspec will not have that problem. I'm not sure what the WebKit policy is for temporary patches like that, but it would be nice to get it accepted, since it helps a lot in the intermediate stage we're in.
Eric Seidel (no email)
Comment 5
2010-12-12 02:44:40 PST
Comment on
attachment 70249
[details]
Patch rs=me.
WebKit Commit Bot
Comment 6
2010-12-12 02:59:56 PST
Comment on
attachment 70249
[details]
Patch Rejecting
attachment 70249
[details]
from commit-queue. Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-cq-sl', 'apply-attachment', '--non-interactive', 70249]" exit_code: 2 Last 500 characters of output: rocess.pro patching file WebKitTools/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file WebKitTools/DumpRenderTree/qt/DumpRenderTree.pro patching file WebKitTools/MiniBrowser/qt/MiniBrowser.pro patching file WebKitTools/QtTestBrowser/QtTestBrowser.pro patching file WebKitTools/WebKitTestRunner/qt/WebKitTestRunner.pro Hunk #1 succeeded at 54 (offset 4 lines). Failed to run "[u'/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Eric Seidel', u'--force']" exit_code: 1 Full output:
http://queues.webkit.org/results/7038005
Kristian Amlie
Comment 7
2010-12-14 02:51:49 PST
This issue is a bit old, and the linux-armcc name is going away anyway. I think it can be safely closed. I'll open a new one if necessary.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug