xenconfig: move contents to libxl driver and remove directory

[libvirt] [PATCH] xenconfig: move contents to libxl driver and remove directory

Posted by Jim Fehlig 4 years, 7 months ago

Signed-off-by: Jim Fehlig <jfehlig@suse.com>
---
 cfg.mk                                   |  2 +-
 configure.ac                             |  2 --
 po/POTFILES                              |  6 ++---
 src/Makefile.am                          |  1 -
 src/libvirt_xenconfig.syms               | 12 ----------
 src/libxl/Makefile.inc.am                | 25 ++++++---------------
 src/{xenconfig => libxl}/xen_common.c    |  0
 src/{xenconfig => libxl}/xen_common.h    |  0
 src/{xenconfig => libxl}/xen_xl.c        |  0
 src/{xenconfig => libxl}/xen_xl.h        |  0
 src/{xenconfig => libxl}/xen_xm.c        |  0
 src/{xenconfig => libxl}/xen_xm.h        |  0
 src/{xenconfig => libxl}/xenxs_private.h |  0
 src/xenconfig/Makefile.inc.am            | 28 ------------------------
 tests/xlconfigtest.c                     |  2 +-
 tests/xmconfigtest.c                     |  2 +-
 16 files changed, 13 insertions(+), 67 deletions(-)

diff --git a/cfg.mk b/cfg.mk
index c459ad443f..1f29729949 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -791,7 +791,7 @@ sc_prohibit_cross_inclusion:
 	    access/ | conf/) safe="($$dir|conf|util)";; \
 	    cpu/| network/| node_device/| rpc/| security/| storage/) \
 	      safe="($$dir|util|conf|storage)";; \
-	    xenapi/ | xenconfig/ ) safe="($$dir|util|conf|xen|cpu)";; \
+	    xenapi/) safe="($$dir|util|conf|xen|cpu)";; \
 	    *) safe="($$dir|$(mid_dirs)|util)";; \
 	  esac; \
 	  in_vc_files="^src/$$dir" \
diff --git a/configure.ac b/configure.ac
index a60543072d..890702a89d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -465,8 +465,6 @@ LIBVIRT_DRIVER_CHECK_LIBVIRTD
 LIBVIRT_DRIVER_CHECK_NETWORK
 LIBVIRT_DRIVER_CHECK_INTERFACE
 
-AM_CONDITIONAL([WITH_XENCONFIG], [test "$with_libxl" = "yes"])
-
 
 dnl
 dnl in case someone want to build static binaries
diff --git a/po/POTFILES b/po/POTFILES
index c62bc32bb2..e466e1bc55 100644
--- a/po/POTFILES
+++ b/po/POTFILES
@@ -88,6 +88,9 @@ src/libxl/libxl_conf.c
 src/libxl/libxl_domain.c
 src/libxl/libxl_driver.c
 src/libxl/libxl_migration.c
+src/libxl/xen_common.c
+src/libxl/xen_xl.c
+src/libxl/xen_xm.c
 src/locking/lock_daemon.c
 src/locking/lock_daemon_dispatch.c
 src/locking/lock_driver_lockd.c
@@ -291,9 +294,6 @@ src/vz/vz_utils.c
 src/vz/vz_utils.h
 src/xenapi/xenapi_driver.c
 src/xenapi/xenapi_utils.c
-src/xenconfig/xen_common.c
-src/xenconfig/xen_xl.c
-src/xenconfig/xen_xm.c
 tests/virpolkittest.c
 tools/libvirt-guests.sh.in
 tools/virsh-checkpoint.c
diff --git a/src/Makefile.am b/src/Makefile.am
index adaf61350a..6626659113 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -118,7 +118,6 @@ include vbox/Makefile.inc.am
 include openvz/Makefile.inc.am
 include qemu/Makefile.inc.am
 include bhyve/Makefile.inc.am
-include xenconfig/Makefile.inc.am
 include libxl/Makefile.inc.am
 include xenapi/Makefile.inc.am
 include vz/Makefile.inc.am
diff --git a/src/libvirt_xenconfig.syms b/src/libvirt_xenconfig.syms
deleted file mode 100644
index 6e2e578b15..0000000000
--- a/src/libvirt_xenconfig.syms
+++ /dev/null
@@ -1,12 +0,0 @@
-#
-# These symbols are dependent upon --with-xen via WITH_XEN or --with-libxl via WITH_LIBXL.
-#
-
-# xenconfig/xen_xm.h
-xenFormatXM;
-xenParseXM;
-
-# Let emacs know we want case-insensitive sorting
-# Local Variables:
-# sort-fold-case: t
-# End:
diff --git a/src/libxl/Makefile.inc.am b/src/libxl/Makefile.inc.am
index 1587404586..560b214877 100644
--- a/src/libxl/Makefile.inc.am
+++ b/src/libxl/Makefile.inc.am
@@ -1,6 +1,13 @@
 # vim: filetype=automake
 
 LIBXL_DRIVER_SOURCES = \
+	libxl/xenxs_private.h \
+	libxl/xen_common.c \
+	libxl/xen_common.h \
+	libxl/xen_xl.c \
+	libxl/xen_xl.h \
+	libxl/xen_xm.c \
+	libxl/xen_xm.h \
 	libxl/libxl_conf.c \
 	libxl/libxl_conf.h \
 	libxl/libxl_capabilities.c \
@@ -15,29 +22,13 @@ LIBXL_DRIVER_SOURCES = \
 	libxl/libxl_migration.h \
 	$(NULL)
 
-XENCONFIG_LIBXL_SOURCES = \
-	$(XENCONFIG_SOURCES) \
-	xenconfig/xen_xl.c \
-	xenconfig/xen_xl.h \
-	$(NULL)
-
 DRIVER_SOURCE_FILES += $(LIBXL_DRIVER_SOURCES)
 STATEFUL_DRIVER_SOURCE_FILES += $(LIBXL_DRIVER_SOURCES)
 EXTRA_DIST += \
 	$(LIBXL_DRIVER_SOURCES) \
-	$(XENCONFIG_LIBXL_SOURCES) \
 	$(NULL)
 
 if WITH_LIBXL
-noinst_LTLIBRARIES += libvirt_xenconfig_libxl.la
-libvirt_xenconfig_libxl_la_LIBADD = $(LIBXL_LIBS)
-libvirt_xenconfig_libxl_la_CFLAGS = \
-	-I$(srcdir)/conf \
-	-I$(srcdir)/libxl \
-	$(AM_CFLAGS) \
-	$(NULL)
-libvirt_xenconfig_libxl_la_SOURCES = $(XENCONFIG_LIBXL_SOURCES)
-
 noinst_LTLIBRARIES += libvirt_driver_libxl_impl.la
 libvirt_driver_libxl_la_SOURCES =
 libvirt_driver_libxl_la_LIBADD = \
@@ -54,14 +45,12 @@ libvirt_driver_libxl_impl_la_CFLAGS = \
 	-I$(srcdir)/access \
 	-I$(srcdir)/conf \
 	-I$(srcdir)/secret \
-	-I$(srcdir)/xenconfig \
 	$(AM_CFLAGS) \
 	$(NULL)
 libvirt_driver_libxl_impl_la_LDFLAGS = $(AM_LDFLAGS)
 libvirt_driver_libxl_impl_la_LIBADD = \
 	$(LIBXL_LIBS) \
 	$(LIBXML_LIBS) \
-	libvirt_xenconfig_libxl.la \
 	$(NULL)
 libvirt_driver_libxl_impl_la_SOURCES = $(LIBXL_DRIVER_SOURCES)
 
diff --git a/src/xenconfig/xen_common.c b/src/libxl/xen_common.c
similarity index 100%
rename from src/xenconfig/xen_common.c
rename to src/libxl/xen_common.c
diff --git a/src/xenconfig/xen_common.h b/src/libxl/xen_common.h
similarity index 100%
rename from src/xenconfig/xen_common.h
rename to src/libxl/xen_common.h
diff --git a/src/xenconfig/xen_xl.c b/src/libxl/xen_xl.c
similarity index 100%
rename from src/xenconfig/xen_xl.c
rename to src/libxl/xen_xl.c
diff --git a/src/xenconfig/xen_xl.h b/src/libxl/xen_xl.h
similarity index 100%
rename from src/xenconfig/xen_xl.h
rename to src/libxl/xen_xl.h
diff --git a/src/xenconfig/xen_xm.c b/src/libxl/xen_xm.c
similarity index 100%
rename from src/xenconfig/xen_xm.c
rename to src/libxl/xen_xm.c
diff --git a/src/xenconfig/xen_xm.h b/src/libxl/xen_xm.h
similarity index 100%
rename from src/xenconfig/xen_xm.h
rename to src/libxl/xen_xm.h
diff --git a/src/xenconfig/xenxs_private.h b/src/libxl/xenxs_private.h
similarity index 100%
rename from src/xenconfig/xenxs_private.h
rename to src/libxl/xenxs_private.h
diff --git a/src/xenconfig/Makefile.inc.am b/src/xenconfig/Makefile.inc.am
deleted file mode 100644
index 5785b49268..0000000000
--- a/src/xenconfig/Makefile.inc.am
+++ /dev/null
@@ -1,28 +0,0 @@
-# vim: filetype=automake
-
-XENCONFIG_SOURCES = \
-	xenconfig/xenxs_private.h \
-	xenconfig/xen_common.c \
-	xenconfig/xen_common.h \
-	xenconfig/xen_xm.c \
-	xenconfig/xen_xm.h \
-	$(NULL)
-
-EXTRA_DIST += $(XENCONFIG_SOURCES)
-
-if WITH_XENCONFIG
-noinst_LTLIBRARIES += libvirt_xenconfig.la
-libvirt_la_BUILT_LIBADD += libvirt_xenconfig.la
-libvirt_xenconfig_la_CFLAGS = \
-	-I$(srcdir)/conf \
-	$(AM_CFLAGS) \
-	$(NULL)
-libvirt_xenconfig_la_SOURCES = $(XENCONFIG_SOURCES)
-
-endif WITH_XENCONFIG
-
-if WITH_XENCONFIG
-USED_SYM_FILES += $(srcdir)/libvirt_xenconfig.syms
-else ! WITH_XENCONFIG
-SYM_FILES += $(srcdir)/libvirt_xenconfig.syms
-endif ! WITH_XENCONFIG
diff --git a/tests/xlconfigtest.c b/tests/xlconfigtest.c
index 59171606d1..ae0db71293 100644
--- a/tests/xlconfigtest.c
+++ b/tests/xlconfigtest.c
@@ -27,7 +27,7 @@
 
 #include "internal.h"
 #include "datatypes.h"
-#include "xenconfig/xen_xl.h"
+#include "libxl/xen_xl.h"
 #include "viralloc.h"
 #include "virstring.h"
 #include "testutils.h"
diff --git a/tests/xmconfigtest.c b/tests/xmconfigtest.c
index 1de3c33f69..3137dc564c 100644
--- a/tests/xmconfigtest.c
+++ b/tests/xmconfigtest.c
@@ -25,7 +25,7 @@
 
 #include "internal.h"
 #include "datatypes.h"
-#include "xenconfig/xen_xm.h"
+#include "libxl/xen_xm.h"
 #include "testutils.h"
 #include "testutilsxen.h"
 #include "viralloc.h"
-- 
2.22.0


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

Re: [libvirt] [PATCH] xenconfig: move contents to libxl driver and remove directory

Posted by Ján Tomko 4 years, 7 months ago

The 'From:' field shows your e-mail in uppercase.

On Fri, Aug 23, 2019 at 07:50:12PM +0000, Jim Fehlig wrote:
>Signed-off-by: Jim Fehlig <jfehlig@suse.com>
>---
> cfg.mk                                   |  2 +-
> configure.ac                             |  2 --
> po/POTFILES                              |  6 ++---
> src/Makefile.am                          |  1 -
> src/libvirt_xenconfig.syms               | 12 ----------
> src/libxl/Makefile.inc.am                | 25 ++++++---------------
> src/{xenconfig => libxl}/xen_common.c    |  0
> src/{xenconfig => libxl}/xen_common.h    |  0
> src/{xenconfig => libxl}/xen_xl.c        |  0
> src/{xenconfig => libxl}/xen_xl.h        |  0
> src/{xenconfig => libxl}/xen_xm.c        |  0
> src/{xenconfig => libxl}/xen_xm.h        |  0
> src/{xenconfig => libxl}/xenxs_private.h |  0
> src/xenconfig/Makefile.inc.am            | 28 ------------------------
> tests/xlconfigtest.c                     |  2 +-
> tests/xmconfigtest.c                     |  2 +-
> 16 files changed, 13 insertions(+), 67 deletions(-)
>

There's one occurrence left:

diff --git a/src/xenapi/Makefile.inc.am b/src/xenapi/Makefile.inc.am
index b6e4ed0bdb..aedef4e2a0 100644
--- a/src/xenapi/Makefile.inc.am
+++ b/src/xenapi/Makefile.inc.am
@@ -18,7 +18,6 @@ libvirt_driver_xenapi_la_CFLAGS = \
        $(XENAPI_CFLAGS) \
        $(CURL_CFLAGS) \
        -I$(srcdir)/conf \
-       -I$(srcdir)/xenconfig \
        $(AM_CFLAGS) \
        $(NULL)
 libvirt_driver_xenapi_la_LDFLAGS = $(AM_LDFLAGS)

With that fixed:
Reviewed-by: Ján Tomko <jtomko@redhat.com>

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

Re: [libvirt] [PATCH] xenconfig: move contents to libxl driver and remove directory

Posted by Jim Fehlig 4 years, 7 months ago

On 8/26/19 5:49 AM, Ján Tomko  wrote:
> The 'From:' field shows your e-mail in uppercase.

Thanks for pointing that out! Seems the O365 server is doing it on my behalf 
:-(. I've entered an internal ticket.

> 
> On Fri, Aug 23, 2019 at 07:50:12PM +0000, Jim Fehlig wrote:
>> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
>> ---
>> cfg.mk                                   |  2 +-
>> configure.ac                             |  2 --
>> po/POTFILES                              |  6 ++---
>> src/Makefile.am                          |  1 -
>> src/libvirt_xenconfig.syms               | 12 ----------
>> src/libxl/Makefile.inc.am                | 25 ++++++---------------
>> src/{xenconfig => libxl}/xen_common.c    |  0
>> src/{xenconfig => libxl}/xen_common.h    |  0
>> src/{xenconfig => libxl}/xen_xl.c        |  0
>> src/{xenconfig => libxl}/xen_xl.h        |  0
>> src/{xenconfig => libxl}/xen_xm.c        |  0
>> src/{xenconfig => libxl}/xen_xm.h        |  0
>> src/{xenconfig => libxl}/xenxs_private.h |  0
>> src/xenconfig/Makefile.inc.am            | 28 ------------------------
>> tests/xlconfigtest.c                     |  2 +-
>> tests/xmconfigtest.c                     |  2 +-
>> 16 files changed, 13 insertions(+), 67 deletions(-)
>>
> 
> There's one occurrence left:
> 
> diff --git a/src/xenapi/Makefile.inc.am b/src/xenapi/Makefile.inc.am
> index b6e4ed0bdb..aedef4e2a0 100644
> --- a/src/xenapi/Makefile.inc.am
> +++ b/src/xenapi/Makefile.inc.am
> @@ -18,7 +18,6 @@ libvirt_driver_xenapi_la_CFLAGS = \
>         $(XENAPI_CFLAGS) \
>         $(CURL_CFLAGS) \
>         -I$(srcdir)/conf \
> -       -I$(srcdir)/xenconfig \
>         $(AM_CFLAGS) \
>         $(NULL)
> libvirt_driver_xenapi_la_LDFLAGS = $(AM_LDFLAGS)
> 
> With that fixed:
> Reviewed-by: Ján Tomko <jtomko@redhat.com>

I removed the other occurrence and pushed the patch.

Regards,
Jim

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

Re: [libvirt] [PATCH] xenconfig: move contents to libxl driver and remove directory

Posted by Michal Prívozník 4 years, 7 months ago

On 8/26/19 1:49 PM, Ján Tomko wrote:
> The 'From:' field shows your e-mail in uppercase.
> 
> On Fri, Aug 23, 2019 at 07:50:12PM +0000, Jim Fehlig wrote:
>> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
>> ---
>> cfg.mk                                   |  2 +-
>> configure.ac                             |  2 --
>> po/POTFILES                              |  6 ++---
>> src/Makefile.am                          |  1 -
>> src/libvirt_xenconfig.syms               | 12 ----------
>> src/libxl/Makefile.inc.am                | 25 ++++++---------------
>> src/{xenconfig => libxl}/xen_common.c    |  0
>> src/{xenconfig => libxl}/xen_common.h    |  0
>> src/{xenconfig => libxl}/xen_xl.c        |  0
>> src/{xenconfig => libxl}/xen_xl.h        |  0
>> src/{xenconfig => libxl}/xen_xm.c        |  0
>> src/{xenconfig => libxl}/xen_xm.h        |  0
>> src/{xenconfig => libxl}/xenxs_private.h |  0
>> src/xenconfig/Makefile.inc.am            | 28 ------------------------
>> tests/xlconfigtest.c                     |  2 +-
>> tests/xmconfigtest.c                     |  2 +-
>> 16 files changed, 13 insertions(+), 67 deletions(-)
>>
> 

Actually, this breaks --with-xenapi build:

xenapi/xenapi_driver.c:38:10: fatal error: xen_common.h: No such file or
directory
 #include "xen_common.h"
          ^~~~~~~~~~~~~~
compilation terminated.

Michal

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

Re: [libvirt] [PATCH] xenconfig: move contents to libxl driver and remove directory

Posted by Jim Fehlig 4 years, 7 months ago

On 8/31/19 2:11 AM, Michal Prívozník  wrote:
> On 8/26/19 1:49 PM, Ján Tomko wrote:
>> The 'From:' field shows your e-mail in uppercase.
>>
>> On Fri, Aug 23, 2019 at 07:50:12PM +0000, Jim Fehlig wrote:
>>> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
>>> ---
>>> cfg.mk                                   |  2 +-
>>> configure.ac                             |  2 --
>>> po/POTFILES                              |  6 ++---
>>> src/Makefile.am                          |  1 -
>>> src/libvirt_xenconfig.syms               | 12 ----------
>>> src/libxl/Makefile.inc.am                | 25 ++++++---------------
>>> src/{xenconfig => libxl}/xen_common.c    |  0
>>> src/{xenconfig => libxl}/xen_common.h    |  0
>>> src/{xenconfig => libxl}/xen_xl.c        |  0
>>> src/{xenconfig => libxl}/xen_xl.h        |  0
>>> src/{xenconfig => libxl}/xen_xm.c        |  0
>>> src/{xenconfig => libxl}/xen_xm.h        |  0
>>> src/{xenconfig => libxl}/xenxs_private.h |  0
>>> src/xenconfig/Makefile.inc.am            | 28 ------------------------
>>> tests/xlconfigtest.c                     |  2 +-
>>> tests/xmconfigtest.c                     |  2 +-
>>> 16 files changed, 13 insertions(+), 67 deletions(-)
>>>
>>
> 
> Actually, this breaks --with-xenapi build:
> 
> xenapi/xenapi_driver.c:38:10: fatal error: xen_common.h: No such file or
> directory
>   #include "xen_common.h"
>            ^~~~~~~~~~~~~~
> compilation terminated.

Can you determine why this is needed? E.g. remove it and see what subsequently 
fails? I don't have a setup readily available to test a xenapi build (I'm not 
"in the office" today as it is a US holiday).

Regards,
Jim

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

Re: [libvirt] [PATCH] xenconfig: move contents to libxl driver and remove directory

Posted by Daniel P. Berrangé 4 years, 7 months ago

On Mon, Sep 02, 2019 at 02:07:08PM +0000, Jim Fehlig wrote:
> On 8/31/19 2:11 AM, Michal Prívozník  wrote:
> > On 8/26/19 1:49 PM, Ján Tomko wrote:
> >> The 'From:' field shows your e-mail in uppercase.
> >>
> >> On Fri, Aug 23, 2019 at 07:50:12PM +0000, Jim Fehlig wrote:
> >>> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
> >>> ---
> >>> cfg.mk                                   |  2 +-
> >>> configure.ac                             |  2 --
> >>> po/POTFILES                              |  6 ++---
> >>> src/Makefile.am                          |  1 -
> >>> src/libvirt_xenconfig.syms               | 12 ----------
> >>> src/libxl/Makefile.inc.am                | 25 ++++++---------------
> >>> src/{xenconfig => libxl}/xen_common.c    |  0
> >>> src/{xenconfig => libxl}/xen_common.h    |  0
> >>> src/{xenconfig => libxl}/xen_xl.c        |  0
> >>> src/{xenconfig => libxl}/xen_xl.h        |  0
> >>> src/{xenconfig => libxl}/xen_xm.c        |  0
> >>> src/{xenconfig => libxl}/xen_xm.h        |  0
> >>> src/{xenconfig => libxl}/xenxs_private.h |  0
> >>> src/xenconfig/Makefile.inc.am            | 28 ------------------------
> >>> tests/xlconfigtest.c                     |  2 +-
> >>> tests/xmconfigtest.c                     |  2 +-
> >>> 16 files changed, 13 insertions(+), 67 deletions(-)
> >>>
> >>
> > 
> > Actually, this breaks --with-xenapi build:
> > 
> > xenapi/xenapi_driver.c:38:10: fatal error: xen_common.h: No such file or
> > directory
> >   #include "xen_common.h"
> >            ^~~~~~~~~~~~~~
> > compilation terminated.
> 
> Can you determine why this is needed? E.g. remove it and see what subsequently 
> fails? I don't have a setup readily available to test a xenapi build (I'm not 
> "in the office" today as it is a US holiday).

I'm inclined to say that we should call the XenAPI driver dead at
this point. IIUC, the person at Citrix who wrote left, left that
job shortly after it was merged into libvirt. Looking at the git
history I'm struggling to see any single patch applied in 9 years
since that was not a general cleanup/bugfix. The XML handling does
not deal with disks at all AFAICT making it largely useless.

IOW, lets just document that it fails to build for this release
and then delete its code.

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

Re: [libvirt] [PATCH] xenconfig: move contents to libxl driver and remove directory

Posted by Jim Fehlig 4 years, 7 months ago

On 9/2/19 8:16 AM, Daniel P. Berrangé  wrote:
> On Mon, Sep 02, 2019 at 02:07:08PM +0000, Jim Fehlig wrote:
>> On 8/31/19 2:11 AM, Michal Prívozník  wrote:
>>> On 8/26/19 1:49 PM, Ján Tomko wrote:
>>>> The 'From:' field shows your e-mail in uppercase.
>>>>
>>>> On Fri, Aug 23, 2019 at 07:50:12PM +0000, Jim Fehlig wrote:
>>>>> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
>>>>> ---
>>>>> cfg.mk                                   |  2 +-
>>>>> configure.ac                             |  2 --
>>>>> po/POTFILES                              |  6 ++---
>>>>> src/Makefile.am                          |  1 -
>>>>> src/libvirt_xenconfig.syms               | 12 ----------
>>>>> src/libxl/Makefile.inc.am                | 25 ++++++---------------
>>>>> src/{xenconfig => libxl}/xen_common.c    |  0
>>>>> src/{xenconfig => libxl}/xen_common.h    |  0
>>>>> src/{xenconfig => libxl}/xen_xl.c        |  0
>>>>> src/{xenconfig => libxl}/xen_xl.h        |  0
>>>>> src/{xenconfig => libxl}/xen_xm.c        |  0
>>>>> src/{xenconfig => libxl}/xen_xm.h        |  0
>>>>> src/{xenconfig => libxl}/xenxs_private.h |  0
>>>>> src/xenconfig/Makefile.inc.am            | 28 ------------------------
>>>>> tests/xlconfigtest.c                     |  2 +-
>>>>> tests/xmconfigtest.c                     |  2 +-
>>>>> 16 files changed, 13 insertions(+), 67 deletions(-)
>>>>>
>>>>
>>>
>>> Actually, this breaks --with-xenapi build:
>>>
>>> xenapi/xenapi_driver.c:38:10: fatal error: xen_common.h: No such file or
>>> directory
>>>    #include "xen_common.h"
>>>             ^~~~~~~~~~~~~~
>>> compilation terminated.
>>
>> Can you determine why this is needed? E.g. remove it and see what subsequently
>> fails? I don't have a setup readily available to test a xenapi build (I'm not
>> "in the office" today as it is a US holiday).
> 
> I'm inclined to say that we should call the XenAPI driver dead at
> this point.

As an alternative to fixing the bugld, I was just about to suggest the same! 
It's a drastic fix: kill the driver :-). I can work on this, perhaps in the 
evening. It's probably not much effort. The driver is not even mentioned in the 
drivers page

https://libvirt.org/drivers.html

Regards,
Jim

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

Re: [libvirt] [PATCH] xenconfig: move contents to libxl driver and remove directory

Posted by Michal Privoznik 4 years, 7 months ago

On 9/2/19 4:16 PM, Daniel P. Berrangé wrote:
> On Mon, Sep 02, 2019 at 02:07:08PM +0000, Jim Fehlig wrote:
>> On 8/31/19 2:11 AM, Michal Prívozník  wrote:
>>> On 8/26/19 1:49 PM, Ján Tomko wrote:
>>>> The 'From:' field shows your e-mail in uppercase.
>>>>
>>>> On Fri, Aug 23, 2019 at 07:50:12PM +0000, Jim Fehlig wrote:
>>>>> Signed-off-by: Jim Fehlig <jfehlig@suse.com>
>>>>> ---
>>>>> cfg.mk                                   |  2 +-
>>>>> configure.ac                             |  2 --
>>>>> po/POTFILES                              |  6 ++---
>>>>> src/Makefile.am                          |  1 -
>>>>> src/libvirt_xenconfig.syms               | 12 ----------
>>>>> src/libxl/Makefile.inc.am                | 25 ++++++---------------
>>>>> src/{xenconfig => libxl}/xen_common.c    |  0
>>>>> src/{xenconfig => libxl}/xen_common.h    |  0
>>>>> src/{xenconfig => libxl}/xen_xl.c        |  0
>>>>> src/{xenconfig => libxl}/xen_xl.h        |  0
>>>>> src/{xenconfig => libxl}/xen_xm.c        |  0
>>>>> src/{xenconfig => libxl}/xen_xm.h        |  0
>>>>> src/{xenconfig => libxl}/xenxs_private.h |  0
>>>>> src/xenconfig/Makefile.inc.am            | 28 ------------------------
>>>>> tests/xlconfigtest.c                     |  2 +-
>>>>> tests/xmconfigtest.c                     |  2 +-
>>>>> 16 files changed, 13 insertions(+), 67 deletions(-)
>>>>>
>>>>
>>>
>>> Actually, this breaks --with-xenapi build:
>>>
>>> xenapi/xenapi_driver.c:38:10: fatal error: xen_common.h: No such file or
>>> directory
>>>    #include "xen_common.h"
>>>             ^~~~~~~~~~~~~~
>>> compilation terminated.
>>
>> Can you determine why this is needed? E.g. remove it and see what subsequently
>> fails? I don't have a setup readily available to test a xenapi build (I'm not
>> "in the office" today as it is a US holiday).
> 
> I'm inclined to say that we should call the XenAPI driver dead at
> this point. IIUC, the person at Citrix who wrote left, left that
> job shortly after it was merged into libvirt. Looking at the git
> history I'm struggling to see any single patch applied in 9 years
> since that was not a general cleanup/bugfix. The XML handling does
> not deal with disks at all AFAICT making it largely useless.
> 
> IOW, lets just document that it fails to build for this release
> and then delete its code.

Agreed.

Michal

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