All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-qt5][PATCH] nativesdk-qtbase: use runtime linked dbus
@ 2016-02-14 16:25 Samuli Piippo
  2016-02-15 22:34 ` Otavio Salvador
  0 siblings, 1 reply; 6+ messages in thread
From: Samuli Piippo @ 2016-02-14 16:25 UTC (permalink / raw)
  To: openembedded-devel

Remove direct dependency to nativesdk-dbus. Instead configure
qtbase to use runtime linked dbus library, if that is found.

Signed-off-by: Samuli Piippo <samuli.piippo@theqtcompany.com>
---
 recipes-qt/qt5/nativesdk-qtbase_git.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/recipes-qt/qt5/nativesdk-qtbase_git.bb b/recipes-qt/qt5/nativesdk-qtbase_git.bb
index 4ac80be..d800b8c 100644
--- a/recipes-qt/qt5/nativesdk-qtbase_git.bb
+++ b/recipes-qt/qt5/nativesdk-qtbase_git.bb
@@ -1,5 +1,5 @@
 DESCRIPTION = "SDK version of Qt/[X11|Mac|Embedded]"
-DEPENDS = "nativesdk-zlib nativesdk-dbus qtbase-native"
+DEPENDS = "nativesdk-zlib qtbase-native"
 SECTION = "libs"
 HOMEPAGE = "http://qt-project.org"
 
@@ -155,6 +155,7 @@ do_configure() {
         -sysroot ${STAGING_DIR_NATIVE} \
         -no-gcc-sysroot \
         -system-zlib \
+        -dbus-runtime \
         -no-libjpeg \
         -no-libpng \
         -no-gif \
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [meta-qt5][PATCH] nativesdk-qtbase: use runtime linked dbus
  2016-02-14 16:25 [meta-qt5][PATCH] nativesdk-qtbase: use runtime linked dbus Samuli Piippo
@ 2016-02-15 22:34 ` Otavio Salvador
  2016-02-16  9:13   ` Samuli Piippo
  0 siblings, 1 reply; 6+ messages in thread
From: Otavio Salvador @ 2016-02-15 22:34 UTC (permalink / raw)
  To: OpenEmbedded Devel List

On Sun, Feb 14, 2016 at 2:25 PM, Samuli Piippo
<samuli.piippo@theqtcompany.com> wrote:
> Remove direct dependency to nativesdk-dbus. Instead configure
> qtbase to use runtime linked dbus library, if that is found.
>
> Signed-off-by: Samuli Piippo <samuli.piippo@theqtcompany.com>

Why? This puts a dependency on the host operating system version.

Am I missing something?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-qt5][PATCH] nativesdk-qtbase: use runtime linked dbus
  2016-02-15 22:34 ` Otavio Salvador
@ 2016-02-16  9:13   ` Samuli Piippo
  2016-02-16 11:38     ` Otavio Salvador
  0 siblings, 1 reply; 6+ messages in thread
From: Samuli Piippo @ 2016-02-16  9:13 UTC (permalink / raw)
  To: openembedded-devel

On 16.02.2016 00:34, Otavio Salvador wrote:
> On Sun, Feb 14, 2016 at 2:25 PM, Samuli Piippo
> <samuli.piippo@theqtcompany.com> wrote:
>> Remove direct dependency to nativesdk-dbus. Instead configure
>> qtbase to use runtime linked dbus library, if that is found.
>>
>> Signed-off-by: Samuli Piippo <samuli.piippo@theqtcompany.com>
>
> Why? This puts a dependency on the host operating system version.
>
> Am I missing something?
>

Currently nativesdk-qtbase does not package libQt5DBus.so, so there 
nothing in the SDK that actual uses dbus, but it's still built as a 
dependency.
If "qt5: add Qml support to lupdate" patch is merged, then 
nativesdk-dbus would be pulled into SDK as well, although I think it's 
still unlikely that anyone would need it. If there is really need for 
it, then you can always add nativesdk-dbus to your toolchain.


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-qt5][PATCH] nativesdk-qtbase: use runtime linked dbus
  2016-02-16  9:13   ` Samuli Piippo
@ 2016-02-16 11:38     ` Otavio Salvador
  2016-02-16 14:13       ` Samuli Piippo
  0 siblings, 1 reply; 6+ messages in thread
From: Otavio Salvador @ 2016-02-16 11:38 UTC (permalink / raw)
  To: OpenEmbedded Devel List

On Tue, Feb 16, 2016 at 7:13 AM, Samuli Piippo
<samuli.piippo@theqtcompany.com> wrote:
> On 16.02.2016 00:34, Otavio Salvador wrote:
>>
>> On Sun, Feb 14, 2016 at 2:25 PM, Samuli Piippo
>> <samuli.piippo@theqtcompany.com> wrote:
>>>
>>> Remove direct dependency to nativesdk-dbus. Instead configure
>>> qtbase to use runtime linked dbus library, if that is found.
>>>
>>> Signed-off-by: Samuli Piippo <samuli.piippo@theqtcompany.com>
>>
>>
>> Why? This puts a dependency on the host operating system version.
>>
>> Am I missing something?
>>
>
> Currently nativesdk-qtbase does not package libQt5DBus.so, so there nothing
> in the SDK that actual uses dbus, but it's still built as a dependency.
> If "qt5: add Qml support to lupdate" patch is merged, then nativesdk-dbus
> would be pulled into SDK as well, although I think it's still unlikely that
> anyone would need it. If there is really need for it, then you can always
> add nativesdk-dbus to your toolchain.

So why not remove libQt5Dbus.so at all, from the nativesdk?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-qt5][PATCH] nativesdk-qtbase: use runtime linked dbus
  2016-02-16 11:38     ` Otavio Salvador
@ 2016-02-16 14:13       ` Samuli Piippo
  2016-02-16 16:01         ` Otavio Salvador
  0 siblings, 1 reply; 6+ messages in thread
From: Samuli Piippo @ 2016-02-16 14:13 UTC (permalink / raw)
  To: openembedded-devel

On 16.02.2016 13:38, Otavio Salvador wrote:
> On Tue, Feb 16, 2016 at 7:13 AM, Samuli Piippo
> <samuli.piippo@theqtcompany.com> wrote:
>> On 16.02.2016 00:34, Otavio Salvador wrote:
>>>
>>> On Sun, Feb 14, 2016 at 2:25 PM, Samuli Piippo
>>> <samuli.piippo@theqtcompany.com> wrote:
>>>>
>>>> Remove direct dependency to nativesdk-dbus. Instead configure
>>>> qtbase to use runtime linked dbus library, if that is found.
>>>>
>>>> Signed-off-by: Samuli Piippo <samuli.piippo@theqtcompany.com>
>>>
>>>
>>> Why? This puts a dependency on the host operating system version.
>>>
>>> Am I missing something?
>>>
>>
>> Currently nativesdk-qtbase does not package libQt5DBus.so, so there nothing
>> in the SDK that actual uses dbus, but it's still built as a dependency.
>> If "qt5: add Qml support to lupdate" patch is merged, then nativesdk-dbus
>> would be pulled into SDK as well, although I think it's still unlikely that
>> anyone would need it. If there is really need for it, then you can always
>> add nativesdk-dbus to your toolchain.
>
> So why not remove libQt5Dbus.so at all, from the nativesdk?
>

Point I'm trying make is, let's not depend on and build nativesdk-dbus 
if we don't need it.
On nativesdk-qtbase, we want to enable dbus, because that gives us also 
Qt dbus tools (which don't need libdbus).


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-qt5][PATCH] nativesdk-qtbase: use runtime linked dbus
  2016-02-16 14:13       ` Samuli Piippo
@ 2016-02-16 16:01         ` Otavio Salvador
  0 siblings, 0 replies; 6+ messages in thread
From: Otavio Salvador @ 2016-02-16 16:01 UTC (permalink / raw)
  To: OpenEmbedded Devel List

On Tue, Feb 16, 2016 at 12:13 PM, Samuli Piippo <samuli.piippo@gmail.com> wrote:
> On 16.02.2016 13:38, Otavio Salvador wrote:
>>
>> On Tue, Feb 16, 2016 at 7:13 AM, Samuli Piippo
>> <samuli.piippo@theqtcompany.com> wrote:
>>>
>>> On 16.02.2016 00:34, Otavio Salvador wrote:
>>>>
>>>>
>>>> On Sun, Feb 14, 2016 at 2:25 PM, Samuli Piippo
>>>> <samuli.piippo@theqtcompany.com> wrote:
>>>>>
>>>>>
>>>>> Remove direct dependency to nativesdk-dbus. Instead configure
>>>>> qtbase to use runtime linked dbus library, if that is found.
>>>>>
>>>>> Signed-off-by: Samuli Piippo <samuli.piippo@theqtcompany.com>
>>>>
>>>>
>>>>
>>>> Why? This puts a dependency on the host operating system version.
>>>>
>>>> Am I missing something?
>>>>
>>>
>>> Currently nativesdk-qtbase does not package libQt5DBus.so, so there
>>> nothing
>>> in the SDK that actual uses dbus, but it's still built as a dependency.
>>> If "qt5: add Qml support to lupdate" patch is merged, then nativesdk-dbus
>>> would be pulled into SDK as well, although I think it's still unlikely
>>> that
>>> anyone would need it. If there is really need for it, then you can always
>>> add nativesdk-dbus to your toolchain.
>>
>>
>> So why not remove libQt5Dbus.so at all, from the nativesdk?
>>
>
> Point I'm trying make is, let's not depend on and build nativesdk-dbus if we
> don't need it.
> On nativesdk-qtbase, we want to enable dbus, because that gives us also Qt
> dbus tools (which don't need libdbus).

What will happen if the host version of the libdbus does not match the
one used during the build?

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-02-16 16:01 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-14 16:25 [meta-qt5][PATCH] nativesdk-qtbase: use runtime linked dbus Samuli Piippo
2016-02-15 22:34 ` Otavio Salvador
2016-02-16  9:13   ` Samuli Piippo
2016-02-16 11:38     ` Otavio Salvador
2016-02-16 14:13       ` Samuli Piippo
2016-02-16 16:01         ` Otavio Salvador

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.