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

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

 



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@xxxxxxxx>
>>> ---
>>> 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@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