[meta-virtualization] [RFC] xen: Break volatiles out into separate package.

[Chris Patterson]

Looks good to me.


On Thu, Dec 5, 2013 at 7:09 PM, Bruce Ashfield <bruce.ashfield at gmail.com>wrote:

> On Tue, Dec 3, 2013 at 9:20 PM, Philip Tricca <flihp at twobit.us> wrote:
> > The Xen scripts for handling block and vif udev events depend on
> > directories created by the Xen volatile directories. In a situation
> > where the 'base' Xen package isn't installed (NDVM etc) these
> > scripts fail silently. The most simple approach I could come up
> > with is just breaking the Xen volatiles config out into it's own
> > package and setting up the logical dependencies.
> >
> > Creating a new package for just a single file seems like overkill
> > to me but it's the most straightforward thing I could come up with.
>
> This seems reasonable to me, there are plenty of other one file package
> examples (in particular since the update-rc.d only works for one initscript
> per package), so that isn't all that out of the ordinary.
>
> So to my eye, this looks ok.
>
> Bruce
>
> >
> > Regards,
> > Philip
> >
> > Signed-off-by: Philip Tricca <flihp at twobit.us>
> > ---
> >  recipes-extended/xen/xen.inc |    8 +++++++-
> >  1 file changed, 7 insertions(+), 1 deletion(-)
> >
> > diff --git a/recipes-extended/xen/xen.inc b/recipes-extended/xen/xen.inc
> > index 9d43814..e31e996 100644
> > --- a/recipes-extended/xen/xen.inc
> > +++ b/recipes-extended/xen/xen.inc
> > @@ -53,12 +53,14 @@ RDEPENDS_${PN}-base = "\
> >  RDEPENDS_${PN}-scripts-block = "\
> >      ${PN}-scripts-common \
> >      ${PN}-udev \
> > +    ${PN}-volatiles \
> >      "
> >
> >  RDEPENDS_${PN}-scripts-network = "\
> >      bridge-utils \
> >      ${PN}-scripts-common \
> >      ${PN}-udev \
> > +    ${PN}-volatiles \
> >      "
> >
> >  PACKAGES = "\
> > @@ -106,6 +108,7 @@ PACKAGES = "\
> >      ${PN}-scripts-network \
> >      ${PN}-staticdev \
> >      ${PN}-udev \
> > +    ${PN}-volatiles \
> >      ${PN}-xcutils \
> >      ${PN}-xencommons \
> >      ${PN}-xend \
> > @@ -210,7 +213,6 @@ FILES_${PN}-hypervisor = "\
> >      "
> >
> >  FILES_${PN}-base = "\
> > -    ${sysconfdir}/default/volatiles/99_xen \
> >      ${sysconfdir}/default/xencommons \
> >      ${sysconfdir}/default/xendomains \
> >      ${sysconfdir}/xen/auto \
> > @@ -425,6 +427,10 @@ FILES_${PN}-udev = "\
> >      ${sysconfdir}/udev/rules.d/xend.rules \
> >      "
> >
> > +FILES_${PN}-volatiles = "\
> > +    ${sysconfdir}/default/volatiles/99_xen \
> > +    "
> > +
> >  FILES_${PN}-xcutils = "\
> >      ${libdir}/xen/bin/lsevtchn \
> >      ${libdir}/xen/bin/readnotes \
> > --
> > 1.7.10.4
> >
> > _______________________________________________
> > meta-virtualization mailing list
> > meta-virtualization at yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-virtualization
>
>
>
> --
> "Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end"
> _______________________________________________
> meta-virtualization mailing list
> meta-virtualization at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-virtualization
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/meta-virtualization/attachments/20131205/b893f05d/attachment.html>