Re: [PATCH 05/12] volatiles.bbclass: add recipe

Richard Purdie
2013-01-24T11:53:19Z

On Thu, 2013-01-24 at 11:31 +0100, Martin Jansa wrote:
> On Thu, Jan 24, 2013 at 04:16:45PM +0800, Qi.Chen@windriver.com wrote:
> > From: Chen Qi <Qi.Chen@windriver.com>
> >
> > All recipes that need to tweak their directories need to inherit this
> > class and set VOLATILE_DIR, VOLATILE_LINK and VOLATILE_PKGS.
> >
> > [YOCTO #3406]
> >
> > Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> > ---
> > meta/classes/volatiles.bbclass | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> > create mode 100644 meta/classes/volatiles.bbclass
> >
> > diff --git a/meta/classes/volatiles.bbclass b/meta/classes/volatiles.bbclass
> > new file mode 100644
> > index 0000000..f27e598
> > --- /dev/null
> > +++ b/meta/classes/volatiles.bbclass
> > @@ -0,0 +1,11 @@
> > +# This bbclass is for read-only-rootfs support. All recipes that may tweak their
> > +# own directories in order to run correctly in a read-only rootfs enviroment
> > +# should inherit this class and set VOLATILE_DIR, VOLATILE_LINK, VOLATILE_PKGS.
> > +
> > +# For each package, the net result is a read-only specific conf file generated
> > +# and packaged, which has the name of readonly-${VOLATILESPN}.
> > +
> > +# All these config files are then parsed by populate-volatile.sh to create proper
> > +# directories and links.
> > +
> > +inherit ${@base_contains('DISTRO_FEATURES','sysvinit','volatiles_sysvinit','',d)}
>
> Does "inherit " work?
>
> I mean empty inherit when DISTRO_FEATURES don't contain sysvinit? IIRC
> it will fail to parse.

The above syntax does now work. A plain "inherit " won't but something
with an expression in does get handled correctly. It did not used to
work with older bitbake versions.

Cheers,

Richard