On Tue, Mar 7, 2017 at 2:22 PM, Andreas Müller
<schnitzelt...@googlemail.com> wrote:
> On Fri, Feb 10, 2017 at 11:19 AM, Andreas Müller
> <schnitzelt...@googlemail.com> wrote:
>> Hi,
>>
>> I am cross building Qt and KDE with yocto environment. After Update to
>> Qt 5.8 the package baloo from KDE KF5 is broken. After investigation I
>> found that qdbuscpp2xml 5.8 build misses some public slots.
>>
>> I get for Qt5.8 build by yocto:
>>
>> qdbuscpp2xml -a filecontentindexer.h
>> Unregistered input type in parameter list: QDBusMessage
>> Unregistered input type in parameter list: QDBusMessage
>> <!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection
>> 1.0//EN" "http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd";>
>> <node>
>>   <interface name="org.kde.baloo.fileindexer">
>>     <property name="currentFile" type="s" access="read"/>
>>     <signal name="startedIndexingFile">
>>       <arg name="filePath" type="s" direction="out"/>
>>     </signal>
>>     <signal name="finishedIndexingFile">
>>       <arg name="filePath" type="s" direction="out"/>
>>     </signal>
>>   </interface>
>> </node>
>>
>> On my local fedora Qt5.7.1
>> qdbuscpp2xml -a filecontentindexer.h
>> <!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection
>> 1.0//EN" "http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd";>
>> <node>
>>   <interface name="org.kde.baloo.fileindexer">
>>     <property name="currentFile" type="s" access="read"/>
>>     <signal name="startedIndexingFile">
>>       <arg name="filePath" type="s" direction="out"/>
>>     </signal>
>>     <signal name="finishedIndexingFile">
>>       <arg name="filePath" type="s" direction="out"/>
>>     </signal>
>>     <method name="registerMonitor">
>>     </method>
>>     <method name="unregisterMonitor">
>>     </method>
>>   </interface>
>> </node>
>>
>> You can see that qdbuscpp2xml complains for unregistered QDBusMessage
>> and the slots 'registerMonitor' and 'unregisterMonitor' having
>> QDBusMessage in parameter are missing.
>>
>> Since yocto builds also native qdbuscpp2xml it might be possible that
>> there is something missing for qdbuscpp2xml.
>>
>> Pointers how to get around or even confirmation of a bug are welcome -
>> baloo's filecontentindexer.h is attached
>>
> Coming back to the original qdbuscpp2xml / baloo problem: I have
> checked further and think to know what happened:
>
> * qdbuscpp2xml build based on Qt5.7 simply ignored QDBusMessage
> parameters. This seemed to be silently accepted by baloo as the call
> to unregisterMonitor was done without parameters [1].
> * qdbuscpp2xml build based on Qt5.8 bootstrapped complained with
> 'Unregistered input type in parameter list: QDBusMessage' and ignoring
> registerMonitor/unregisterMonitor as mentioned above.
> * qdbuscpp2xml build based on Qt5.8 and patched (attachment) to
> non-bootstrapped handles QDBusMessage properly.
>
> I dropped a note a kde-devel either but can not find it in any
> archive. Anyway I can hacked baloo to build now and am happy.
>
> [1] https://cgit.kde.org/baloo.git/tree/src/qml/experimental/monitor.cpp#n114
>
Forgot attachment

Andreas
From 86ae0ce931edbe0ac30cfe5f94f681f1d055fdb0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20M=C3=BCller?= <schnitzelt...@googlemail.com>
Date: Mon, 6 Mar 2017 11:50:38 +0100
Subject: [PATCH] qdbuscpp2xml.pro: do not build with bootstrapped
 dependenciies
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Upstream-Status: Pending

Signed-off-by: Andreas Müller <schnitzelt...@googlemail.com>
---
 src/src.pro                             | 9 ++-------
 src/tools/qdbuscpp2xml/qdbuscpp2xml.pro | 3 +--
 src/tools/qdbusxml2cpp/qdbusxml2cpp.pro | 5 +----
 3 files changed, 4 insertions(+), 13 deletions(-)

diff --git a/src/src.pro b/src/src.pro
index 90d7e2b..0aaf604 100644
--- a/src/src.pro
+++ b/src/src.pro
@@ -4,9 +4,6 @@ QT_FOR_CONFIG += gui-private
 include($$OUT_PWD/corelib/qtcore-config.pri)
 include($$OUT_PWD/gui/qtgui-config.pri)
 
-force_bootstrap|!qtConfig(commandlineparser): \
-    CONFIG += force_dbus_bootstrap
-
 src_qtzlib.file = $$PWD/corelib/qtzlib.pro
 src_qtzlib.target = sub-zlib
 
@@ -37,13 +34,11 @@ src_tools_bootstrap_dbus.depends = src_tools_bootstrap
 
 src_tools_qdbusxml2cpp.subdir = tools/qdbusxml2cpp
 src_tools_qdbusxml2cpp.target = sub-qdbusxml2cpp
-force_dbus_bootstrap: src_tools_qdbusxml2cpp.depends = src_tools_bootstrap_dbus
-else: src_tools_qdbusxml2cpp.depends = src_dbus
+src_tools_qdbusxml2cpp.depends = src_dbus
 
 src_tools_qdbuscpp2xml.subdir = tools/qdbuscpp2xml
 src_tools_qdbuscpp2xml.target = sub-qdbuscpp2xml
-force_bootstrap: src_tools_qdbuscpp2xml.depends = src_tools_bootstrap_dbus
-else: src_tools_qdbuscpp2xml.depends = src_dbus
+src_tools_qdbuscpp2xml.depends = src_dbus
 
 src_winmain.subdir = $$PWD/winmain
 src_winmain.target = sub-winmain
diff --git a/src/tools/qdbuscpp2xml/qdbuscpp2xml.pro b/src/tools/qdbuscpp2xml/qdbuscpp2xml.pro
index 5ac3159..74c507c 100644
--- a/src/tools/qdbuscpp2xml/qdbuscpp2xml.pro
+++ b/src/tools/qdbuscpp2xml/qdbuscpp2xml.pro
@@ -1,7 +1,6 @@
 option(host_build)
 QT = core-private
-force_bootstrap: QT += bootstrap_dbus-private
-else: QT += dbus-private
+QT += dbus-private
 DEFINES += QT_NO_CAST_FROM_ASCII QT_NO_FOREACH
 QMAKE_CXXFLAGS += $$QT_HOST_CFLAGS_DBUS
 
diff --git a/src/tools/qdbusxml2cpp/qdbusxml2cpp.pro b/src/tools/qdbusxml2cpp/qdbusxml2cpp.pro
index d9ee5de..e891103 100644
--- a/src/tools/qdbusxml2cpp/qdbusxml2cpp.pro
+++ b/src/tools/qdbusxml2cpp/qdbusxml2cpp.pro
@@ -1,9 +1,6 @@
 option(host_build)
-!force_bootstrap:!qtConfig(commandlineparser): \
-    CONFIG += force_bootstrap
 QT = core-private
-force_bootstrap: QT += bootstrap_dbus-private
-else: QT += dbus-private
+QT += dbus-private
 DEFINES += QT_NO_CAST_FROM_ASCII QT_NO_FOREACH
 QMAKE_CXXFLAGS += $$QT_HOST_CFLAGS_DBUS
 
-- 
2.9.3

_______________________________________________
Development mailing list
Development@qt-project.org
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to