[libvirt] [PATCH 0/2] Remove xenapi driver

Jim Fehlig posted 2 patches 4 years, 7 months ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20190903045805.12185-1-jfehlig@suse.com
cfg.mk                             |    4 +-
configure.ac                       |    4 -
docs/aclpolkit.html.in             |    4 -
docs/auth.html.in                  |    2 -
docs/news.xml                      |   10 +
libvirt.spec.in                    |    4 +-
m4/virt-driver-xenapi.m4           |   48 -
po/POTFILES                        |    2 -
src/Makefile.am                    |    1 -
src/README                         |    1 -
src/libvirt.c                      |   10 -
src/util/virrandom.c               |    3 +-
src/xenapi/Makefile.inc.am         |   29 -
src/xenapi/xenapi_driver.c         | 2125 ----------------------------
src/xenapi/xenapi_driver.h         |   22 -
src/xenapi/xenapi_driver_private.h |   58 -
src/xenapi/xenapi_utils.c          |  562 --------
src/xenapi/xenapi_utils.h          |   76 -
tools/virsh.c                      |    3 -
19 files changed, 13 insertions(+), 2955 deletions(-)
delete mode 100644 m4/virt-driver-xenapi.m4
delete mode 100644 src/xenapi/Makefile.inc.am
delete mode 100644 src/xenapi/xenapi_driver.c
delete mode 100644 src/xenapi/xenapi_driver.h
delete mode 100644 src/xenapi/xenapi_driver_private.h
delete mode 100644 src/xenapi/xenapi_utils.c
delete mode 100644 src/xenapi/xenapi_utils.h
[libvirt] [PATCH 0/2] Remove xenapi driver
Posted by Jim Fehlig 4 years, 7 months ago
Remove the xenapi driver is unmaintained and has no known user base.
We are stuck with references in include/libvirt/virterror.h and
src/util/virerror.c but otherwise I think I've nuked xenapi everywhere.

Jim Fehlig (2):
  xenapi: remove driver
  news: Mention removal of xenapi driver

 cfg.mk                             |    4 +-
 configure.ac                       |    4 -
 docs/aclpolkit.html.in             |    4 -
 docs/auth.html.in                  |    2 -
 docs/news.xml                      |   10 +
 libvirt.spec.in                    |    4 +-
 m4/virt-driver-xenapi.m4           |   48 -
 po/POTFILES                        |    2 -
 src/Makefile.am                    |    1 -
 src/README                         |    1 -
 src/libvirt.c                      |   10 -
 src/util/virrandom.c               |    3 +-
 src/xenapi/Makefile.inc.am         |   29 -
 src/xenapi/xenapi_driver.c         | 2125 ----------------------------
 src/xenapi/xenapi_driver.h         |   22 -
 src/xenapi/xenapi_driver_private.h |   58 -
 src/xenapi/xenapi_utils.c          |  562 --------
 src/xenapi/xenapi_utils.h          |   76 -
 tools/virsh.c                      |    3 -
 19 files changed, 13 insertions(+), 2955 deletions(-)
 delete mode 100644 m4/virt-driver-xenapi.m4
 delete mode 100644 src/xenapi/Makefile.inc.am
 delete mode 100644 src/xenapi/xenapi_driver.c
 delete mode 100644 src/xenapi/xenapi_driver.h
 delete mode 100644 src/xenapi/xenapi_driver_private.h
 delete mode 100644 src/xenapi/xenapi_utils.c
 delete mode 100644 src/xenapi/xenapi_utils.h

-- 
2.22.0


--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/2] Remove xenapi driver
Posted by Daniel P. Berrangé 4 years, 7 months ago
On Tue, Sep 03, 2019 at 04:58:23AM +0000, Jim Fehlig wrote:
> Remove the xenapi driver is unmaintained and has no known user base.
> We are stuck with references in include/libvirt/virterror.h and
> src/util/virerror.c but otherwise I think I've nuked xenapi everywhere.

Yes, that's OK, we must not change the XML parsers or public APIs.

> 
> Jim Fehlig (2):
>   xenapi: remove driver
>   news: Mention removal of xenapi driver
> 
>  cfg.mk                             |    4 +-
>  configure.ac                       |    4 -
>  docs/aclpolkit.html.in             |    4 -
>  docs/auth.html.in                  |    2 -
>  docs/news.xml                      |   10 +
>  libvirt.spec.in                    |    4 +-
>  m4/virt-driver-xenapi.m4           |   48 -
>  po/POTFILES                        |    2 -
>  src/Makefile.am                    |    1 -
>  src/README                         |    1 -
>  src/libvirt.c                      |   10 -
>  src/util/virrandom.c               |    3 +-
>  src/xenapi/Makefile.inc.am         |   29 -
>  src/xenapi/xenapi_driver.c         | 2125 ----------------------------
>  src/xenapi/xenapi_driver.h         |   22 -
>  src/xenapi/xenapi_driver_private.h |   58 -
>  src/xenapi/xenapi_utils.c          |  562 --------
>  src/xenapi/xenapi_utils.h          |   76 -
>  tools/virsh.c                      |    3 -
>  19 files changed, 13 insertions(+), 2955 deletions(-)
>  delete mode 100644 m4/virt-driver-xenapi.m4
>  delete mode 100644 src/xenapi/Makefile.inc.am
>  delete mode 100644 src/xenapi/xenapi_driver.c
>  delete mode 100644 src/xenapi/xenapi_driver.h
>  delete mode 100644 src/xenapi/xenapi_driver_private.h
>  delete mode 100644 src/xenapi/xenapi_utils.c
>  delete mode 100644 src/xenapi/xenapi_utils.h

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

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