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:[email protected]] > Sent: Thursday, September 25, 2014 8:14 PM > To: Josep Puigdemont > Cc: Yin Olivia-R63875; [email protected] > Subject: Re: [meta-virtualization] [PATCH] libvirt: uprev to v1.2.8 > > On Thu, Sep 25, 2014 at 6:38 AM, Josep Puigdemont > <[email protected]> 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 > >> [email protected] > >> https://lists.yoctoproject.org/listinfo/meta-virtualization > > -- > > _______________________________________________ > > meta-virtualization mailing list > > [email protected] > > 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 [email protected] https://lists.yoctoproject.org/listinfo/meta-virtualization
