Bug 4575

Summary: qt-mobility: qmetaobjectbuilder fails to build
Product: [Build System, Metadata & Runtime] OE-Core Reporter: Wenzong.Fan <wenzong.fan>
Component: coreAssignee: Hongxu Jia <hongxu.jia>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: bluelightning, liezhi.yang, Martin.Jansa, meta.mr.watcher, meta.watcher, sgw, wenzong.fan
Version: 1.4   
Target Milestone: 1.6 M2   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Wenzong.Fan 2013-05-30 02:44:17 UTC 
This is a known issue on upstream:
https://bugreports.qt-project.org/browse/QTMOBILITY-1990?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel

And has been fixed by:
http://qt.gitorious.org/qt-mobility/qt-mobility/commit/f102053b28009b3094b0e5777177208afa6097c5

Reproduce steps in Yocto:
1) Add below lines to local.conf:
# Remove '-fpermissive' option from CXXFLAGS
export BUILD_CXXFLAGS = "${BUILD_CFLAGS}"
export TARGET_CXXFLAGS = "${TARGET_CFLAGS}"

2) bitbake qt-mobility-x11 && bitbake qt-mobility-embedded

3) Error messages:
ipc/qmetaobjectbuilder.cpp:803:65: error: invalid conversion from 'QMetaObjectExtraData::StaticMetacallFunction {aka void (*)(QObject*, QMetaObject::Call, int, void**)} ' to 'QtMobility::QMetaObjectBuilder::StaticMetacallFunction {aka int (*)(QMetaObject::Call, int, void**)} 

We should backport the upstream patches to Yocto.
Comment 1 Hongxu Jia 2013-06-28 09:38:55 UTC 
If keep '-fpermissive' option, the above diagnostics about
nonconformant code from errors to warnings as following:
...
ipc/qmetaobjectbuilder.cpp:803:65: warning: invalid conversion from
'QMetaObjectExtraData::StaticMetacallFunction {aka void (*)(QObject*,
QMetaObject::Call, int, void**)}' to 'QtMobility::QMetaObjectBuilder
::StaticMetacallFunction {aka int (*)(QMetaObject::Call, int, void*
*)}' [-fpermissive]
...

Hi Saul,

Should we fix the conversion warning in this situation? Or just keep
option '-fpermissive' as where it is.

If choose to fix the conversion warning, there is already a patch in
oe-core to fix this:
http://qt.gitorious.org/qt-mobility/qt-mobility/commit/f102053b28009b3094b0e5777177208afa6097c5 

If not, I will close this bug.

Thanks,
Hongxu
Comment 2 Hongxu Jia 2013-06-28 09:42:21 UTC 
> http://qt.gitorious.org/qt-mobility/qt-mobility/commit/
> f102053b28009b3094b0e5777177208afa6097c5 
> 

http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=wenzong/qt-mobility 

//Hongxu
Comment 3 Hongxu Jia 2013-06-29 05:42:10 UTC 
Hi Paul,

What's your suggestions about the above question?

Thanks,
Hongxu
Comment 4 Paul Eggleton 2013-07-02 22:05:53 UTC 
I'm OK with Wenzong's backported patch linked above, although it is a bit large.
Comment 5 Martin Jansa 2014-02-27 09:56:22 UTC 
If you know about qt-mobility, can you please also check:
https://bugzilla.yoctoproject.org/show_bug.cgi?id=5414
Comment 6 Hongxu Jia 2014-03-03 01:50:46 UTC 
Merged into master:

commit ae59f6ff20a6ccbf12bc030f250baffc384042f2
Author: Hongxu Jia <hongxu.jia@windriver.com>
Date:   Thu Feb 27 17:32:19 2014 +0800

    qt-mobility: fix metaobjectbuilder build errors