Re: [PATCH] file: add bzip2-replacement-native to DEPENDS to fix sstate issue

["Richard Purdie" <richard.purdie@linuxfoundation.org>]

On Sun, 2020-05-24 at 12:04 +0200, Gregor Zatko wrote:
> From: Jan-Simon Moeller <dl9pf@gmx.de>
> 
> file-native when built on a Debian 10 host will embed a dependency to
> 'libbz2.so.1.0' (instead of 'libbz2.so.1'). This can cause issues
> when sharing the sstate between hosts e.g.:
> 
>  recipe-sysroot-native/usr/lib/rpm/rpmdeps:
>       error while loading shared libraries: libbz2.so.1.0: \
>         cannot open shared object file: No such file or directory
> 
> To avoid this situation, let's add the bzip2-replacement-native to
> the
> file recipe's DEPENDS_class-native .
> 
> Details in https://bugzilla.yoctoproject.org/show_bug.cgi?id=13915 .
> 
> (From OE-Core rev: 4a996574464028bd5d57b90920d0887d1a81e9e9)
> 
> Signed-off-by: Jan-Simon Moeller <dl9pf@gmx.de>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  meta/classes/sanity.bbclass | 6 ++++++
>  1 file changed, 6 insertions(+)

I think you've mixed up two different patches here, merging Jan-Simon's 
patch with your own inherit one...

Cheers,

Richard

> diff --git a/meta/classes/sanity.bbclass
> b/meta/classes/sanity.bbclass
> index 292c5591dd..e021b9d240 100644
> --- a/meta/classes/sanity.bbclass
> +++ b/meta/classes/sanity.bbclass
> @@ -784,6 +784,12 @@ def check_sanity_everybuild(status, d):
>      if "." in paths or "./" in paths or "" in paths:
>          status.addresult("PATH contains '.', './' or '' (empty
> element), which will break the build, please remove this.\nParsed
> PATH is " + str(paths) + "\n")
>  
> +    # Check whether 'inherit' directive is found (used for a class
> to inherit)
> +    # in conf file it's supposed to be uppercase INHERIT
> +    inherit = d.getVar('inherit')
> +    if inherit:
> +        status.addresult("Please don't use inherit directive in your
> local.conf. The directive is supposed to be used in classes and
> recipes only to inherit of bbclasses. Here INHERIT should be
> used.\n")
> +
>      # Check that the DISTRO is valid, if set
>      # need to take into account DISTRO renaming DISTRO
>      distro = d.getVar('DISTRO')