[OE-core] [PATCH] sstate.bbclass: consider postinstall dependencies too

[Martin Jansa]

Shouldn't this list be defined in layer.conf like
SIGGEN_EXCLUDERECIPES_ABISAFE, so that other layers can extend that list?


On Mon, Mar 4, 2013 at 11:37 AM, Laurentiu Palcu
<laurentiu.palcu at intel.com>wrote:

> When checking the dependencies in setscene_depvalid(), make sure we also
> consider those dependencies needed when running the postinstalls on
> host.
>
> [YOCTO #3918]
>
> Signed-off-by: Laurentiu Palcu <laurentiu.palcu at intel.com>
> ---
>  meta/classes/sstate.bbclass |    8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
> index 118a28a..be29e52 100644
> --- a/meta/classes/sstate.bbclass
> +++ b/meta/classes/sstate.bbclass
> @@ -640,6 +640,10 @@ def setscene_depvalid(task, taskdependees, notneeded,
> d):
>          if x in ["quilt-native", "autoconf-native", "automake-native",
> "gnu-config-native", "libtool-native", "pkgconfig-native", "gcc-cross",
> "binutils-cross", "gcc-cross-initial"]:
>              return True
>          return False
> +    def isPostInstDep(x):
> +        if x in ["qemu-native", "gdk-pixbuf-native", "gtk+-native",
> "qemuwrapper-cross", "depmodwrapper-cross"]:
> +            return True
> +        return False
>
>      # We can skip these "safe" dependencies since the aren't runtime
> dependencies, just build time
>      if isSafeDep(taskdependees[task][0]) and taskdependees[task][1] ==
> "do_populate_sysroot":
> @@ -658,8 +662,10 @@ def setscene_depvalid(task, taskdependees, notneeded,
> d):
>          # do_package_write_* and do_package doesn't need do_package
>          if taskdependees[task][1] == "do_package" and
> taskdependees[dep][1] in ['do_package', 'do_package_write_deb',
> 'do_package_write_ipk', 'do_package_write_rpm', 'do_packagedata']:
>              continue
> -        # do_package_write_* and do_package doesn't need
> do_populate_sysroot
> +        # do_package_write_* and do_package doesn't need
> do_populate_sysroot, unless is a postinstall dependency
>          if taskdependees[task][1] == "do_populate_sysroot" and
> taskdependees[dep][1] in ['do_package', 'do_package_write_deb',
> 'do_package_write_ipk', 'do_package_write_rpm', 'do_packagedata']:
> +            if isPostInstDep(taskdependees[task][0]) and
> taskdependees[dep][1] in ['do_package_write_deb', 'do_package_write_ipk',
> 'do_package_write_rpm']:
> +                return False
>              continue
>          # Native/Cross packages don't exist and are noexec anyway
>          if isNativeCross(taskdependees[dep][0]) and taskdependees[dep][1]
> in ['do_package_write_deb', 'do_package_write_ipk', 'do_package_write_rpm',
> 'do_packagedata']:
> --
> 1.7.9.5
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20130304/2e370c6a/attachment-0002.html>