diff mbox

[Xen-devel,v4,1/3] Move xenstore and libxc public headers to include subdir

Message ID alpine.DEB.2.02.1408131203130.2293@kaball.uk.xensource.com
State New
Headers show

Commit Message

Stefano Stabellini Aug. 13, 2014, 11:37 a.m. UTC
On Thu, 7 Aug 2014, Ian Jackson wrote:
> Stefano Stabellini writes ("Re: [Xen-devel] [PATCH v4 1/3] Move xenstore and libxc public headers to include subdir"):
> > I needed to make a couple of more libxc to libxc/include substitutions
> > under tools/ocaml.
> > The appended patch should work.
> > I have also pushed:
> > 
> > git://xenbits.xen.org/people/sstabellini/xen-unstable.git public_headers_v5
> 
> It seems that the 32-bit stubdom build is still broken:
> 
> In file included from xc_core.c:64:0:
> xg_private.h:30:21: fatal error: xenctrl.h: No such file or directory

Somehow I missed this email until today.

I modified the stubdom and mini-os Makefiles (below is the incremental
diff):




I have also pushed everything:

git://xenbits.xen.org/people/sstabellini/xen-unstable.git public_headers_v6

Comments

Ian Jackson Sept. 22, 2014, 5:14 p.m. UTC | #1
Stefano Stabellini writes ("Re: [Xen-devel] [PATCH v4 1/3] Move xenstore and libxc public headers to include subdir"):
> I have also pushed everything:
> 
> git://xenbits.xen.org/people/sstabellini/xen-unstable.git public_headers_v6

I rebased this (onto e01fa4e223fc077e18936d890dbd4813c3e7c9b1) and git
was a bit grumbly and said it had to do a 3-way merge.  After I did
that, my build still failed - see below.  The runes I use are:

  git-clean -xdff && cp ../.config .
  (./configure && make -j4 && echo ok.) 2>&1 | tee ../log
and my .config is:
  CONFIG_QEMU=/u/iwj/work/1/qemu-iwj.git
  QEMU_UPSTREAM_URL=/u/iwj/work/1/qemu-upstream-unstable.git
  OVMF_UPSTREAM_URL=/u/iwj/work/1/ovmf.git

Thanks,
Ian.

make[3]: Entering directory `/u/iwj/work/xen.git/tools/qemu-xen-traditional-dir'
/u/iwj/work/1/qemu-iwj.git/xen-hooks.mak:63: === pciutils-dev package not found - missing /usr/include/pci
/u/iwj/work/1/qemu-iwj.git/xen-hooks.mak:64: === PCI passthrough capability has been disabled
  CC    qemu-nbd.o
  CC    qemu-tool.o
  CC    osdep.o
  CC    cutils.o
In file included from ./config-host.h:20:0,
                 from /u/iwj/work/1/qemu-iwj.git/osdep.c:25:
/u/iwj/work/1/qemu-iwj.git/xen-config-host.h:19:21: fatal error: xenctrl.h: No such file or directory
compilation terminated.
In file included from ./config-host.h:20:0,
                 from /u/iwj/work/1/qemu-iwj.git/qemu-common.h:33,
                 from /u/iwj/work/1/qemu-iwj.git/qemu-nbd.c:20:
/u/iwj/work/1/qemu-iwj.git/xen-config-host.h:19:21: fatal error: xenctrl.h: No such file or directory
compilation terminated.
make[3]: *** [osdep.o] Error 1
make[3]: *** Waiting for unfinished jobs....
In file included from ./config-host.h:20:0,
                 from /u/iwj/work/1/qemu-iwj.git/qemu-common.h:33,
                 from /u/iwj/work/1/qemu-iwj.git/qemu-tool.c:14:
/u/iwj/work/1/qemu-iwj.git/xen-config-host.h:19:21: fatal error: xenctrl.h: No such file or directory
compilation terminated.
In file included from ./config-host.h:20:0,
                 from /u/iwj/work/1/qemu-iwj.git/qemu-common.h:33,
                 from /u/iwj/work/1/qemu-iwj.git/cutils.c:24:
/u/iwj/work/1/qemu-iwj.git/xen-config-host.h:19:21: fatal error: xenctrl.h: No such file or directory
compilation terminated.
make[3]: *** [qemu-nbd.o] Error 1
make[3]: *** [qemu-tool.o] Error 1
make[3]: *** [cutils.o] Error 1
make[3]: Leaving directory `/u/iwj/work/xen.git/tools/qemu-xen-traditional-dir'
make[2]: *** [subdir-install-qemu-xen-traditional-dir] Error 2
make[2]: Leaving directory `/u/iwj/work/xen.git/tools'
make[1]: *** [subdirs-install] Error 2
make[1]: Leaving directory `/u/iwj/work/xen.git/tools'
make: *** [install-tools] Error 2
Ian Campbell Sept. 22, 2014, 6:52 p.m. UTC | #2
On Mon, 2014-09-22 at 18:14 +0100, Ian Jackson wrote:
> Stefano Stabellini writes ("Re: [Xen-devel] [PATCH v4 1/3] Move xenstore and libxc public headers to include subdir"):
> > I have also pushed everything:
> > 
> > git://xenbits.xen.org/people/sstabellini/xen-unstable.git public_headers_v6
> 
> I rebased this (onto e01fa4e223fc077e18936d890dbd4813c3e7c9b1) and git
> was a bit grumbly and said it had to do a 3-way merge.  After I did
> that, my build still failed - see below. 

Looks like you've not applied the associated qemu-trad.git change to
qemu-iwj.git, patch #2 in this series. As noted in the coverletter they
should be applied together (with the corresponding Config.mk change
folded in here)

>  The runes I use are:
> 
>   git-clean -xdff && cp ../.config .
>   (./configure && make -j4 && echo ok.) 2>&1 | tee ../log
> and my .config is:
>   CONFIG_QEMU=/u/iwj/work/1/qemu-iwj.git
>   QEMU_UPSTREAM_URL=/u/iwj/work/1/qemu-upstream-unstable.git
>   OVMF_UPSTREAM_URL=/u/iwj/work/1/ovmf.git
> 
> Thanks,
> Ian.
> 
> make[3]: Entering directory `/u/iwj/work/xen.git/tools/qemu-xen-traditional-dir'
> /u/iwj/work/1/qemu-iwj.git/xen-hooks.mak:63: === pciutils-dev package not found - missing /usr/include/pci
> /u/iwj/work/1/qemu-iwj.git/xen-hooks.mak:64: === PCI passthrough capability has been disabled
>   CC    qemu-nbd.o
>   CC    qemu-tool.o
>   CC    osdep.o
>   CC    cutils.o
> In file included from ./config-host.h:20:0,
>                  from /u/iwj/work/1/qemu-iwj.git/osdep.c:25:
> /u/iwj/work/1/qemu-iwj.git/xen-config-host.h:19:21: fatal error: xenctrl.h: No such file or directory
> compilation terminated.
> In file included from ./config-host.h:20:0,
>                  from /u/iwj/work/1/qemu-iwj.git/qemu-common.h:33,
>                  from /u/iwj/work/1/qemu-iwj.git/qemu-nbd.c:20:
> /u/iwj/work/1/qemu-iwj.git/xen-config-host.h:19:21: fatal error: xenctrl.h: No such file or directory
> compilation terminated.
> make[3]: *** [osdep.o] Error 1
> make[3]: *** Waiting for unfinished jobs....
> In file included from ./config-host.h:20:0,
>                  from /u/iwj/work/1/qemu-iwj.git/qemu-common.h:33,
>                  from /u/iwj/work/1/qemu-iwj.git/qemu-tool.c:14:
> /u/iwj/work/1/qemu-iwj.git/xen-config-host.h:19:21: fatal error: xenctrl.h: No such file or directory
> compilation terminated.
> In file included from ./config-host.h:20:0,
>                  from /u/iwj/work/1/qemu-iwj.git/qemu-common.h:33,
>                  from /u/iwj/work/1/qemu-iwj.git/cutils.c:24:
> /u/iwj/work/1/qemu-iwj.git/xen-config-host.h:19:21: fatal error: xenctrl.h: No such file or directory
> compilation terminated.
> make[3]: *** [qemu-nbd.o] Error 1
> make[3]: *** [qemu-tool.o] Error 1
> make[3]: *** [cutils.o] Error 1
> make[3]: Leaving directory `/u/iwj/work/xen.git/tools/qemu-xen-traditional-dir'
> make[2]: *** [subdir-install-qemu-xen-traditional-dir] Error 2
> make[2]: Leaving directory `/u/iwj/work/xen.git/tools'
> make[1]: *** [subdirs-install] Error 2
> make[1]: Leaving directory `/u/iwj/work/xen.git/tools'
> make: *** [install-tools] Error 2
Ian Jackson Sept. 23, 2014, 1:15 p.m. UTC | #3
Ian Campbell writes ("Re: [Xen-devel] [PATCH v4 1/3] Move xenstore and libxc public headers to include subdir"):
> Looks like you've not applied the associated qemu-trad.git change to
> qemu-iwj.git, patch #2 in this series. As noted in the coverletter they
> should be applied together (with the corresponding Config.mk change
> folded in here)

You were right.  I have now applied 1/3 and 2/3 together.  (That is, I
have applied 2/3 to qemu trad, and amended 1/3 to contain also a
Config.mk update to mention the new qemu commit.)

I'm about to push 3/3.

Thanks,
Ian.
Stefano Stabellini Sept. 23, 2014, 1:22 p.m. UTC | #4
On Tue, 23 Sep 2014, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [PATCH v4 1/3] Move xenstore and libxc public headers to include subdir"):
> > Looks like you've not applied the associated qemu-trad.git change to
> > qemu-iwj.git, patch #2 in this series. As noted in the coverletter they
> > should be applied together (with the corresponding Config.mk change
> > folded in here)
> 
> You were right.  I have now applied 1/3 and 2/3 together.  (That is, I
> have applied 2/3 to qemu trad, and amended 1/3 to contain also a
> Config.mk update to mention the new qemu commit.)
> 
> I'm about to push 3/3.

Thanks!
diff mbox

Patch

diff --git a/extras/mini-os/Config.mk b/extras/mini-os/Config.mk
index d61877b..4852443 100644
--- a/extras/mini-os/Config.mk
+++ b/extras/mini-os/Config.mk
@@ -46,7 +46,7 @@  DEF_CPPFLAGS += -D__MINIOS__
 ifeq ($(libc),y)
 DEF_CPPFLAGS += -DHAVE_LIBC
 DEF_CPPFLAGS += -isystem $(MINI-OS_ROOT)/include/posix
-DEF_CPPFLAGS += -isystem $(XEN_ROOT)/tools/xenstore
+DEF_CPPFLAGS += -isystem $(XEN_ROOT)/tools/xenstore/include
 endif
 
 ifneq ($(LWIPDIR),)
diff --git a/stubdom/Makefile b/stubdom/Makefile
index 6bea68b..ffebc0d 100644
--- a/stubdom/Makefile
+++ b/stubdom/Makefile
@@ -306,12 +306,14 @@  mk-headers-$(XEN_TARGET_ARCH): $(IOEMU_LINKFARM_TARGET)
 	mkdir -p libxc-$(XEN_TARGET_ARCH)
 	[ -h libxc-$(XEN_TARGET_ARCH)/Makefile ] || ( cd libxc-$(XEN_TARGET_ARCH) && \
 	  ln -sf $(XEN_ROOT)/tools/libxc/*.h . && \
+	  ln -sf $(XEN_ROOT)/tools/libxc/include/*.h . && \
 	  ln -sf $(XEN_ROOT)/tools/libxc/*.c . && \
 	  ln -sf $(XEN_ROOT)/tools/libxc/Makefile . )
 	mkdir -p xenstore
 	[ -h xenstore/Makefile ] || ( cd xenstore && \
 	  ln -sf $(XEN_ROOT)/tools/xenstore/*.c . && \
 	  ln -sf $(XEN_ROOT)/tools/xenstore/*.h . && \
+	  ln -sf $(XEN_ROOT)/tools/xenstore/include/*.h . && \
 	  ln -sf $(XEN_ROOT)/tools/xenstore/Makefile . )
 	$(MAKE) DESTDIR= -C $(MINI_OS) links
 	touch mk-headers-$(XEN_TARGET_ARCH)
diff --git a/stubdom/grub/Makefile b/stubdom/grub/Makefile
index 6bd2c4c..934cc4c 100644
--- a/stubdom/grub/Makefile
+++ b/stubdom/grub/Makefile
@@ -5,7 +5,7 @@  vpath %.c ../grub-upstream
 
 BOOT=$(OBJ_DIR)/boot-$(XEN_TARGET_ARCH).o
 
-DEF_CPPFLAGS += -I$(XEN_ROOT)/tools/libxc -I$(XEN_ROOT)/tools/include -I.
+DEF_CPPFLAGS += -I$(XEN_ROOT)/tools/libxc/include -I$(XEN_ROOT)/tools/include -I.
 DEF_CPPFLAGS += -I../grub-upstream/stage1
 DEF_CPPFLAGS += -I../grub-upstream/stage2
 DEF_CPPFLAGS += -I../grub-upstream/netboot