Re: [PATCH 3/3] libxl: Change virDomainObjIsActive to virDomainObjCheckIsActive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/24/2017 01:03 AM, Sagar Ghuge wrote:
Replace virDomainObjIsActive with virDomainObjCheckIsActive
which raises error if domain is not active

Signed-off-by: Sagar Ghuge <ghugesss@xxxxxxxxx>
---
 ...libxl-Implement-virDomainObjCheckIsActive.patch |  55 ++++
 ...omainObjIsActive-with-virDomainObjCheckIs.patch | 109 ++++++++
 ...e-virDomainObjIsActive-to-virDomainObjChe.patch | 298 +++++++++++++++++++++
 patch1                                             |  52 ++++
 src/libxl/libxl_driver.c                           | 102 ++-----
 5 files changed, 537 insertions(+), 79 deletions(-)
 create mode 100644 0001-libxl-Implement-virDomainObjCheckIsActive.patch
 create mode 100644 0002-change-virDomainObjIsActive-with-virDomainObjCheckIs.patch
 create mode 100644 0003-libxl-Change-virDomainObjIsActive-to-virDomainObjChe.patch
 create mode 100644 patch1

Mistakenly added some older patches to the patch? :-) I assume you were only wanting to change src/libxl/libxl_driver.c in this patch. Please remove the embedded patches from this patch in the next version.

BTW, 'make syntax-check' fails when run on this patch. Ensure you run 'make syntax-check' and 'make check' on each patch before submitting to the list. Thanks!

Regards,
Jim

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux