Re: [meta-oe][PATCH] lmsensors: allow lmsensors-isatools to be empty

[Martin Jansa <martin.jansa@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1398 bytes --]

On Fri, Nov 20, 2015 at 07:22:04PM +0800, jackie.huang@windriver.com wrote:
> From: Jackie Huang <jackie.huang@windriver.com>
> 
> isadump and isaset are only built on x86 machines,
> so allow lmsensors-isatools to be empty.
> 
> Signed-off-by: Jackie Huang <jackie.huang@windriver.com>
> ---
>  meta-oe/recipes-support/lm_sensors/lmsensors_3.4.0.bb | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/meta-oe/recipes-support/lm_sensors/lmsensors_3.4.0.bb b/meta-oe/recipes-support/lm_sensors/lmsensors_3.4.0.bb
> index 62ea131..364eb9e 100644
> --- a/meta-oe/recipes-support/lm_sensors/lmsensors_3.4.0.bb
> +++ b/meta-oe/recipes-support/lm_sensors/lmsensors_3.4.0.bb
> @@ -131,3 +131,7 @@ RDEPENDS_${PN}-pwmconfig = "${PN}-fancontrol"
>  FILES_${PN}-isatools = "${sbindir}/isa*"
>  FILES_${PN}-isatools-dbg = "${sbindir}/.debug/isa*"
>  FILES_${PN}-isatools-doc = "${mandir}/man8/isa*"
> +
> +# isadump and isaset are only built on x86 machines,
> +# so allow it to be empty.
> +ALLOW_EMPTY_${PN}-isatools = "1"

Why should anyone on !x86 install lmsensors-isatools package?

And if (s)he does, then why should s(he) be happy with empty package?

Package manager saying that the package doesn't exist is much better
user experience than it happily installing completely useless empty
package.

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]