Re: [meta-networking][PATCH 1/1] ypbind-mt: add dependency but keep as broken

[Joe MacDonald <Joe_MacDonald@mentor.com>]

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

[[oe] [meta-networking][PATCH 1/1] ypbind-mt: add dependency but keep as broken] On 15.07.22 (Wed 11:54) Joe Slater wrote:

> We will need the conditional dependency on systemd.  It is
> not a good idea to use PNBLACKLIST in a recipe, and the

It isn't a good idea to have a recipe that needs to be blacklisted.  :-)

That said, it's here and there's still an argument for supporting it, so
why remove PNBLACKLIST and replace it with something less obvious?

No objection to a conditional dependency on systemd.

-J.

> version here does not suppress lib32-ypbind-mt.
> 
> Signed-off-by: Joe Slater <jslater@windriver.com>
> ---
>  recipes-support/nis/ypbind-mt_2.2.bb |    6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-networking/recipes-support/nis/ypbind-mt_2.2.bb b/meta-networking/recipes-support/nis/ypbind-mt_2.2.bb
> index 4f8bf48..359b020 100644
> --- a/meta-networking/recipes-support/nis/ypbind-mt_2.2.bb
> +++ b/meta-networking/recipes-support/nis/ypbind-mt_2.2.bb
> @@ -15,10 +15,12 @@ of known secure NIS server (/etc/yp.conf) Binds to \
>  the server which answered as first. \
>  "
>  HOMEPAGE = "http://www.linux-nis.org/nis/ypbind-mt/index.html"
> -DEPENDS = "yp-tools"
> +DEPENDS = "yp-tools ${@base_contains('DISTRO_FEATURES', 'systemd', 'systemd', '', d)}"
>  PROVIDES += "ypbind"
>  
> -PNBLACKLIST[ypbind-mt] ?= "BROKEN: Depends on broken yp-tools"
> +python () {
> +    raise bb.parse.SkipPackage("BROKEN: Depends on broken yp-tools")
> +}
>  
>  SRC_URI = "http://www.linux-nis.org/download/ypbind-mt/${BP}.tar.bz2 \
>             file://ypbind-yocto.init \
> -- 
> 1.7.9.5
> 
-- 
-Joe MacDonald.
:wq

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