Re: [PATCH] sstate.bbclass: consider postinstall dependencies too

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

On Mon, 2013-03-04 at 12:48 +0100, Martin Jansa wrote:
> On Mon, Mar 04, 2013 at 01:38:33PM +0200, Laurentiu Palcu wrote:
> > 
> > 
> > On 03/04/2013 01:13 PM, Martin Jansa wrote:
> > > On Mon, Mar 04, 2013 at 10:56:12AM +0000, Richard Purdie wrote:
> > >> On Mon, 2013-03-04 at 11:51 +0100, Martin Jansa wrote:
> > >>> Shouldn't this list be defined in layer.conf like
> > >>> SIGGEN_EXCLUDERECIPES_ABISAFE, so that other layers can extend that
> > >>> list?
> > >>
> > >> I'm hoping we only have very small list of these problematic
> > >> dependencies. If we end up with a large number of them then yes, we'd
> > >> need to abstract it but I'd prefer to avoid that unless proven
> > >> necessary.
> > >>
> > >> Is there a specific case you have in mind?
> > > 
> > > Isn't systemd-systemctl-native another candidate to be included?
> > > 
> > It looks like it should be in the list... I'll resend a new version.
> 
> what about all u-a-native implementations?

Its worth noting that not everything has to be in this list. If the
dependencies get pulled into the rootfs construction by some other
means, we don't need the code here. In this case image.bbclass does:

DEPENDS += "${MLPREFIX}qemuwrapper-cross ${MLPREFIX}depmodwrapper-cross"
do_rootfs[depends] += "virtual/update-alternatives-native:do_populate_sysroot update-rc.d-native:do_populate_sysroot"

so those aren't strictly needed in the list.

Cheers,

Richard