Re: [meta-oe][PATCH 2/2][resend] udev: consolekit package requires x11

[Andreas Oberritter <obi@opendreambox.org>]

On 02.03.2012 09:17, Koen Kooi wrote:
> Op 01-03-12 13:20, Andreas Oberritter schreef:
>> * consolekit fails to build without x11, so disable creation of
>> udev-consolekit, if x11 is unavailable.
> 
>> Signed-off-by: Andreas Oberritter <obi@opendreambox.org> --- 
>> meta-oe/recipes-core/udev/udev_173.bb |    2 +- 
>> meta-oe/recipes-core/udev/udev_175.bb |    2 +- 
>> meta-oe/recipes-core/udev/udev_181.bb |    2 +- 3 files changed, 3
>> insertions(+), 3 deletions(-)
> 
>> diff --git a/meta-oe/recipes-core/udev/udev_173.bb
>> b/meta-oe/recipes-core/udev/udev_173.bb index e4e634a..cbbe910 100644 ---
>> a/meta-oe/recipes-core/udev/udev_173.bb +++
>> b/meta-oe/recipes-core/udev/udev_173.bb @@ -59,7 +59,7 @@
>> do_configure_prepend() { INITSCRIPT_NAME = "udev" INITSCRIPT_PARAMS =
>> "start 04 S ."
> 
>> -PACKAGES =+ "${PN}-systemd libudev libgudev udev-utils udev-consolekit" 
>> +PACKAGES =+ "${PN}-systemd libudev libgudev udev-utils
>> ${@base_contains('DISTRO_FEATURES', 'x11', 'udev-consolekit', '', d)}"
> 
> What does this actually accomplish? If the package is empty it won't get
> generated anyway and if it isn't empty you just introduced a QA error.

The recipe contains:

RDEPENDS_${PN}-consolekit += "consolekit"

Therefore, consolekit gets built when building udev, but fails:

ERROR: Nothing RPROVIDES 'consolekit' (but .../meta-oe/recipes-core/udev/udev_181.bb RDEPENDS on or otherwise requires it)
ERROR: consolekit was skipped: 'x11' not in DISTRO_FEATURES
NOTE: Runtime target 'consolekit' is unbuildable, removing...
Missing or unbuildable dependency chain was: ['consolekit']
ERROR: Required build target 'udev' has no buildable providers.
Missing or unbuildable dependency chain was: ['udev', 'consolekit']

That's what the patch is addressing. Which alternative
would you suggest instead? To omit the runtime dependency?

About the QA error, if this is the error you're referring to,
then the warning is not very helpful:

WARNING: For recipe udev, the following files/directories were installed but not shipped in any package:
WARNING:   /usr/sbin

Regards,
Andreas