Re: [libvirt] [PATCH] xen-proxy: Remove it entirely and use libvirtd instead

On Mon, Nov 08, 2010 at 06:35:14PM +0100, Matthias Bolte wrote: > Suggested by danpb, as it's not up-to-date anymore and > lacks many functions that were added to libvirtd. > --- >  Makefile.am                        |    2 +- >  autobuild.sh                       |    3 +- >  cfg.mk                             |    1 - >  configure.ac                       |   26 +- >  docs/uri.html.in                   |    3 +- >  include/libvirt/virterror.h        |    2 +- >  libvirt.spec.in                    |   14 +- >  po/POTFILES.in                     |    1 - >  proxy/.gitignore                   |    5 - >  proxy/Makefile.am                  |   42 -- >  proxy/libvirt_proxy.c              |  856 ----------------------------- >  src/Makefile.am                    |    1 - >  src/conf/cpu_conf.c                |    2 - >  src/conf/cpu_conf.h                |    6 +- >  src/conf/domain_conf.c             |   17 - >  src/conf/domain_conf.h             |    2 - >  src/conf/nwfilter_params.c         |    3 - >  src/conf/storage_encryption_conf.c |    3 - >  src/util/util.c                    |   67 ++-- >  src/xen/proxy_internal.c           | 1044 ------------------------------------ >  src/xen/proxy_internal.h           |  100 ---- >  src/xen/xen_driver.c               |   56 +-- >  src/xen/xen_driver.h               |   20 +- >  src/xen/xen_hypervisor.c           |   34 +- >  src/xen/xen_hypervisor.h           |    1 - >  src/xen/xend_internal.c            |   56 +-- >  src/xen/xs_internal.c              |   78 +--- >  src/xen/xs_internal.h              |    3 - >  28 files changed, 74 insertions(+), 2374 deletions(-) >  delete mode 100644 proxy/.gitignore >  delete mode 100644 proxy/Makefile.am >  delete mode 100644 proxy/libvirt_proxy.c >  delete mode 100644 src/xen/proxy_internal.c >  delete mode 100644 src/xen/proxy_internal.h ACK, it is well overdue to get rid of this code now we've had libvirtd around for 3 years, and any connection to the Xen proxy will *already* also have a connection to libvirtd for its network/storage/nodedev APIs :-) Daniel