dbus: fix a hard dependency about dbus-ptest

Changes since V3:

Add class-target suffix to RDEPENDS_dbus-ptest.

The following changes since commit bb66113bde5361b869dce2bdaece5b938f077ea8:

bitbake: fetch2: Fix bug in file checksum generation (2014-04-06 11:31:26 +0100)

are available in the git repository at:

git://git.pokylinux.org/poky-contrib chonglu/dbus
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=chonglu/dbus

Chong Lu (1):
dbus: fix a hard dependency about dbus-ptest

 meta/recipes-core/dbus/{dbus-ptest_1.6.18.bb => dbus-test_1.6.18.bb} | 0
 meta/recipes-core/dbus/dbus.inc                                      | 4 +++-
 2 files changed, 3 insertions(+), 1 deletion(-)
 rename meta/recipes-core/dbus/{dbus-ptest_1.6.18.bb => dbus-test_1.6.18.bb} (100%)

1.8.1.2

If image contains dbus and ptest is in DISTRO_FEATURES, dbus-ptest package is installed, regardless of whether ptest-pkgs is in IMAGE_FEATURES. This issue will increase size for most small images.
This patch fixes this problem.

[YOCTO #5702]

---
 meta/recipes-core/dbus/{dbus-ptest_1.6.18.bb => dbus-test_1.6.18.bb} | 0
 meta/recipes-core/dbus/dbus.inc                                      | 4 +++-
 2 files changed, 3 insertions(+), 1 deletion(-)
 rename meta/recipes-core/dbus/{dbus-ptest_1.6.18.bb => dbus-test_1.6.18.bb} (100%)

similarity index 100%
rename from meta/recipes-core/dbus/dbus-ptest_1.6.18.bb
rename to meta/recipes-core/dbus/dbus-test_1.6.18.bb

index 5727ae7..9a9a00a 100644
@@ -6,9 +6,11 @@ LICENSE = "AFL-2 | GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=10dded3b58148f3f1fd804b26354af3e \
                     file://dbus/dbus.h;beginline=6;endline=20;md5=7755c9d7abccd5dbd25a6a974538bb3c"
 DEPENDS = "expat virtual/libintl"
-RDEPENDS_dbus = "${@base_contains('DISTRO_FEATURES', 'ptest', 'dbus-ptest-ptest', '', d)}"
 RDEPENDS_dbus_class-native = ""
 RDEPENDS_dbus_class-nativesdk = ""
+PACKAGES += "${@base_contains('DISTRO_FEATURES', 'ptest', 'dbus-ptest', '', d)}"
+ALLOW_EMPTY_dbus-ptest = "1"
+RDEPENDS_dbus-ptest_class-target = "dbus-test-ptest"
 
 SRC_URI = "http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \
            file://tmpdir.patch \

1.8.1.2

ping

Hi all,

I see this was finally included in oe-core *master*, but the problem is still affecting Daisy.

Creating a basic command-line image with dbus pulls in the dbus-ptest-ptest package (37MB installed):

# du -sh /usr/lib/dbus-ptest/ptest
37.6M   /usr/lib/dbus-ptest/ptest

Could the fix be backported to Daisy?

-
Thanks,

Javier Viguera
Software Engineer
Digi International® Spain S.A.U.

If you'd be willing to backport the patch, verify that it works, and send it to the list, then that would be much appreciated and speed integration.

Ross

Hi Ross,

Attached the obvious cherry-pick of the commit from *master* with a basic conflict solved. But I don't think this is acceptable in an already-released stable branch "Daisy" as it involves a recipe rename (dbus-ptest_1.6.18.bb => dbus-test_1.6.18.bb) that probably will break the projects of current users of yocto Daisy.

Instead, would a basic change like the following be enough?:

-RDEPENDS_dbus = "${@base_contains('DISTRO_FEATURES', 'ptest', 'dbus-ptest-ptest', '', d)}" +RDEPENDS_dbus = "${@base_contains('PTEST_ENABLED', '1', 'dbus-ptest-ptest', '', d)}"

This does not change any recipe name so it seems suitable for a bug-fix only branch but unfortunately this goes beyond my skills in Yocto and ptest framework so any advise would be appreciated.

-
Javier Viguera
Software Engineer
Digi International® Spain S.A.U.

From 1395fc67a29b568daae191cc69089bed5124f311 Mon Sep 17 00:00:00 2001
From: Chong Lu <[email protected]>
Date: Tue, 8 Apr 2014 11:30:56 +0800
Subject: [PATCH] dbus: fix a hard dependency about dbus-ptest

If image contains dbus and ptest is in DISTRO_FEATURES, dbus-ptest package is installed, regardless of whether ptest-pkgs is in IMAGE_FEATURES. This issue will increase size for most small images.
This patch fixes this problem.

[YOCTO #5702]

(From OE-Core rev: 0416583f014138656babdf78a574357ae5ff25bd)

Conflicts:
meta/recipes-core/dbus/dbus.inc

---
 meta/recipes-core/dbus/{dbus-ptest_1.6.18.bb => dbus-test_1.6.18.bb} | 0
 meta/recipes-core/dbus/dbus.inc                                      | 4 +++-
 2 files changed, 3 insertions(+), 1 deletion(-)
 rename meta/recipes-core/dbus/{dbus-ptest_1.6.18.bb => dbus-test_1.6.18.bb} (100%)

similarity index 100%
rename from meta/recipes-core/dbus/dbus-ptest_1.6.18.bb
rename to meta/recipes-core/dbus/dbus-test_1.6.18.bb

index 5727ae77b223..9a9a00af371a 100644
@@ -6,9 +6,11 @@ LICENSE = "AFL-2 | GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=10dded3b58148f3f1fd804b26354af3e \
                     file://dbus/dbus.h;beginline=6;endline=20;md5=7755c9d7abccd5dbd25a6a974538bb3c"
 DEPENDS = "expat virtual/libintl"
-RDEPENDS_dbus = "${@base_contains('DISTRO_FEATURES', 'ptest', 'dbus-ptest-ptest', '', d)}"
 RDEPENDS_dbus_class-native = ""
 RDEPENDS_dbus_class-nativesdk = ""
+PACKAGES += "${@base_contains('DISTRO_FEATURES', 'ptest', 'dbus-ptest', '', d)}"
+ALLOW_EMPTY_dbus-ptest = "1"
+RDEPENDS_dbus-ptest_class-target = "dbus-test-ptest"
 
 SRC_URI = "http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \
            file://tmpdir.patch \

That change doesn't solve your problem, so no. I'm not the daisy maintainer but I can see a good argument for merging the change even with the rename.

Ross

+PACKAGES += "${@base_contains('DISTRO_FEATURES', 'ptest', 'dbus-ptest', '', d)}"

This was changed in master to use PTEST_ENABLED:

PACKAGES += "${@bb.utils.contains('PTEST_ENABLED', '1', 'dbus-ptest', '', d)}"

If you could compare dbus.inc in this patch with master to ensure that there's nothing else that should be merged that would be great. Also remove the Conflicts line from the message.

Ross

For what its worth I've pulled the two patches we need into the daisy branch for this. I'd appreciate someone confirming it does the right thing though.

Cheers,

Richard

Hi Richard,

I've just made a basic test and at least the dbus-ptest issue is solved for me. The package was removed from the rootfs image and the overall size went down in about ~38MB, so yeah I'm happy with that.

Thanks,

Javier Viguera