Bug 47414 – [Qt] Get the symbian/linux-armcc compile working properly

• Component: Tools / Tests
• Status: RESOLVED WONTFIX

[Keith Kyzivat]

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.

[Keith Kyzivat]

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]

Created attachment 70249 [details]
Patch

[Keith Kyzivat]

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]

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 on attachment 70249 [details]
Patch

rs=me.

[WebKit Commit Bot]

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]

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.