[libvirt] [PATCH 00/12] Replace Xen xl parsing/formatting impl

Jim Fehlig jfehlig at suse.com
Sat Jan 10 05:03:32 UTC 2015

The first attempt to implement support for parsing/formatting Xen's
xl disk config format copied Xen's flex-based parser into libvirt, which
has proved to be challenging in the context of autotools.  But as it turns
out, Xen provides an interface to the parser via libxlutil.

This series reverts the first attempt, along with subsequent attempts to
fix it, and replaces it with an implementation based on libxlutil.  The
first nine patches revert the original implementation and subsequent fixes.
Patch 10 provides an implemenation based on libxlutil.  Patches 11 and
12 are basically unchanged from patches 3 and 4 in the first attempt.

One upshot of using libxlutil instead of copying the flex source is
removing the potential for source divergence.

Jim Fehlig (10):
  Revert "bootstrap.conf: add check for flex"
  Revert "src/Makefile: Fix parallel build after xen_xl_disk parser
  Revert "src/Makefile: move the new xen_xl_disk parser code at the
    correct place"
  Revert "Revert "src/Makefile.am: fix build breaker for xenconfig""
  Revert "build: fix xenconfig VPATH builds"
  Revert "src/Makefile.am: fix build breaker for xenconfig"
  Revert "libxl: Add support for parsing/formating Xen XL config"
  Revert "tests: Tests for the xen-xl parser"
  Revert "src/xenconfig: Xen-xl parser"
  Introduce support for parsing/formatting Xen xl config format

Kiarie Kahurani (2):
  tests: Tests for the xen-xl parser
  libxl: Add support for parsing/formating Xen XL config

 .gitignore                           |   1 -
 bootstrap.conf                       |   3 +-
 cfg.mk                               |   3 +-
 configure.ac                         |   4 +-
 src/Makefile.am                      |  49 ++-----
 src/xenconfig/xen_xl.c               | 205 ++++++++++++++--------------
 src/xenconfig/xen_xl_disk.l          | 256 -----------------------------------
 src/xenconfig/xen_xl_disk_i.h        |  39 ------
 tests/xlconfigdata/test-new-disk.cfg |   2 +-
 9 files changed, 114 insertions(+), 448 deletions(-)
 delete mode 100644 src/xenconfig/xen_xl_disk.l
 delete mode 100644 src/xenconfig/xen_xl_disk_i.h


More information about the libvir-list mailing list