[meta-virtualization] [PATCH] libvirt: uprev to v1.2.8

[Hong-Hua.Yin at freescale.com]

Sorry, I didn't mean to remove ptest.
I just confuse when this part will be merged to libvirt upstream.
It seemed that there is no conflict with ptest patches to work on libvirt-v1.2.8.
I'll submit the v2 patch to upgrade the version.

Best Regards,
Olivia

> -----Original Message-----
> From: Bruce Ashfield [mailto:bruce.ashfield at gmail.com]
> Sent: Thursday, September 25, 2014 8:14 PM
> To: Josep Puigdemont
> Cc: Yin Olivia-R63875; meta-virtualization at yoctoproject.org
> Subject: Re: [meta-virtualization] [PATCH] libvirt: uprev to v1.2.8
> 
> On Thu, Sep 25, 2014 at 6:38 AM, Josep Puigdemont
> <josep.puigdemont at enea.com> wrote:
> > On Thu, Sep 25, 2014 at 04:21:16PM +0800, Olivia Yin wrote:
> >> ---
> >>  .../libvirt/{libvirt_1.2.4.bb => libvirt_1.2.8.bb}      | 17 +++--------------
> >>  1 file changed, 3 insertions(+), 14 deletions(-)  rename
> >> recipes-extended/libvirt/{libvirt_1.2.4.bb => libvirt_1.2.8.bb} (91%)
> >>
> >> diff --git a/recipes-extended/libvirt/libvirt_1.2.4.bb
> >> b/recipes-extended/libvirt/libvirt_1.2.8.bb
> >> similarity index 91%
> >> rename from recipes-extended/libvirt/libvirt_1.2.4.bb
> >> rename to recipes-extended/libvirt/libvirt_1.2.8.bb
> >> index 56b4d4b..cdcd54b 100644
> >> --- a/recipes-extended/libvirt/libvirt_1.2.4.bb
> >> +++ b/recipes-extended/libvirt/libvirt_1.2.8.bb
> >> @@ -24,16 +24,12 @@ RDEPENDS_libvirt-libvirtd_append_x86 = " dmidecode"
> >>  RCONFLICTS_${PN}_libvirtd = "connman"
> >>
> >>  SRC_URI = "http://libvirt.org/sources/libvirt-${PV}.tar.gz;name=libvirt \
> >> -           file://tools-add-libvirt-net-rpc-to-virt-host-validate-when.patch \
> >>             file://libvirtd.sh \
> >>             file://libvirtd.conf \
> >> -           file://runptest.patch \
> >> -           file://run-ptest \
> >> -           file://tests-allow-separated-src-and-build-dirs.patch \
> >>            "
> >>
> >> -SRC_URI[libvirt.md5sum] = "da7a9ca519df45a460659189fe0024e6"
> >> -SRC_URI[libvirt.sha256sum] =
> "e43ac5f6b2baeafcd01777be03a897e636f8d48c0cdfb4c4cbb80d45faa9e875"
> >> +SRC_URI[libvirt.md5sum] = "75114991290f7c8f01dd5223431b9c00"
> >> +SRC_URI[libvirt.sha256sum] =
> "0049940b013ea5858b53d9b540c9df9cc3c8cb08750acfd96e9991a1f709e100"
> >>
> >>  inherit autotools gettext update-rc.d pkgconfig ptest
> >>
> >> @@ -68,10 +64,6 @@ ac_cv_path_LVS=/usr/sbin/lvs \
> >> ac_cv_path_PARTED=/usr/sbin/parted \
> >> ac_cv_path_DMSETUP=/usr/sbin/dmsetup"
> >>
> >> -# Ensure that libvirt uses polkit rather than policykit, whether the
> >> host has -# pkcheck installed or not, and ensure the path is correct per our
> config.
> >> -CACHED_CONFIGUREVARS +=
> "ac_cv_path_PKCHECK_PATH=${bindir}/pkcheck"
> >> -
> >>  # Some other possible paths we are not yet setting
> >> #ac_cv_path_RPCGEN=  #ac_cv_path_XSLTPROC= @@ -80,6 +72,7 @@
> >> CACHED_CONFIGUREVARS +=
> "ac_cv_path_PKCHECK_PATH=${bindir}/pkcheck"
> >>  #ac_cv_path_EBTABLES_PATH=
> >>  #ac_cv_path_PKG_CONFIG=
> >>  #ac_cv_path_ac_pt_PKG_CONFIG
> >> +#ac_cv_path_PKCHECK_PATH=
> >>  #ac_cv_path_POLKIT_AUTH=
> >>  #ac_cv_path_DTRACE=
> >>  #ac_cv_path_ISCSIADM=
> >> @@ -172,7 +165,6 @@ PACKAGECONFIG[ebtables] =
> >> "ac_cv_path_EBTABLES_PATH=/sbin/ebtables,ac_cv_path_EB
> >>  PACKAGECONFIG[python] = ",,python,"
> >>  PACKAGECONFIG[sasl] = "--with-sasl,--without-sasl,cyrus-sasl,cyrus-sasl"
> >>  PACKAGECONFIG[iproute2] =
> "ac_cv_path_IP_PATH=/sbin/ip,ac_cv_path_IP_PATH=,iproute2,iproute2"
> >> -PACKAGECONFIG[numactl] = "--with-numactl,--without-numactl,numactl,"
> >>  PACKAGECONFIG[fuse] = "--with-fuse,--without-fuse,fuse,"
> >>
> >>  # Enable the Python tool support
> >> @@ -203,9 +195,6 @@ do_install_append() {
> >>            >> ${D}${sysconfdir}/default/volatiles/99_libvirt
> >>       echo "d root root 0755 ${localstatedir}/run/libvirt/qemu none" \
> >>            >> ${D}${sysconfdir}/default/volatiles/99_libvirt
> >> -
> >> -     # Add hook support for libvirt
> >> -     mkdir -p ${D}/etc/libvirt/hooks
> >>  }
> >>
> >>  EXTRA_OEMAKE = "BUILD_DIR=${B} DEST_DIR=${D}${PTEST_PATH}
> PTEST_DIR=${PTEST_PATH}"
> >
> > Apart from the version upgrade, there are many things being removed
> > from the recipe, like patches, ptests, numactl configuration.... could
> > you explain a bit the purpose of the removal?
> 
> Agreed. In particular, I'd quite like to keep the hooks directory creation and
> numactl.
> So unless there's some sort of horrible error that I've never seen, we wouldn't
> merge this as-is.
> 
> Cheers,
> 
> Bruce
> 
> >
> > Thank you,
> >
> > /Josep
> >
> >> --
> >> 2.1.0.27.g96db324
> >>
> >> --
> >> _______________________________________________
> >> meta-virtualization mailing list
> >> meta-virtualization at yoctoproject.org
> >> https://lists.yoctoproject.org/listinfo/meta-virtualization
> > --
> > _______________________________________________
> > 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"