All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] config: generate Paths-$SUBSYSTEM.mk
@ 2015-05-06  5:02 Wei Liu
  2015-05-08 14:43 ` Ian Campbell
  0 siblings, 1 reply; 3+ messages in thread
From: Wei Liu @ 2015-05-06  5:02 UTC (permalink / raw)
  To: xen-devel; +Cc: Ian Jackson, Wei Liu, Ian Campbell

... so that we can run ./configure in each subsystem.

Make inclusion of Paths-*.mk mandatory. Skip generating a global
Paths.mk since it's of no use anymore.

Signed-off-by: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
---
v2: update .gitignore, delete Paths-*.mk in distclean

Please rerun autogen.sh when applying this patch.
---
 .gitignore           | 2 +-
 config/Docs.mk.in    | 2 +-
 config/Stubdom.mk.in | 2 +-
 config/Tools.mk.in   | 2 +-
 configure.ac         | 5 +----
 docs/Makefile        | 3 ++-
 docs/configure.ac    | 5 ++++-
 stubdom/Makefile     | 3 ++-
 stubdom/configure.ac | 5 ++++-
 tools/Makefile       | 3 ++-
 tools/configure.ac   | 1 +
 11 files changed, 20 insertions(+), 13 deletions(-)

diff --git a/.gitignore b/.gitignore
index c6185a0..8381422 100644
--- a/.gitignore
+++ b/.gitignore
@@ -37,7 +37,7 @@ config.log
 config.status
 config.cache
 config/Toplevel.mk
-config/Paths.mk
+config/Paths*.mk
 
 build-*
 dist/*
diff --git a/config/Docs.mk.in b/config/Docs.mk.in
index 33bd529..df86d93 100644
--- a/config/Docs.mk.in
+++ b/config/Docs.mk.in
@@ -1,4 +1,4 @@
--include $(XEN_ROOT)/config/Paths.mk
+include $(XEN_ROOT)/config/Paths-docs.mk
 
 # Tools
 FIG2DEV             := @FIG2DEV@
diff --git a/config/Stubdom.mk.in b/config/Stubdom.mk.in
index 5990fc4..ea8700c 100644
--- a/config/Stubdom.mk.in
+++ b/config/Stubdom.mk.in
@@ -1,4 +1,4 @@
--include $(XEN_ROOT)/config/Paths.mk
+include $(XEN_ROOT)/config/Paths-stubdom.mk
 
 # Path Programs
 CMAKE               := @CMAKE@
diff --git a/config/Tools.mk.in b/config/Tools.mk.in
index e7da99d..bf6c4bc 100644
--- a/config/Tools.mk.in
+++ b/config/Tools.mk.in
@@ -1,4 +1,4 @@
--include $(XEN_ROOT)/config/Paths.mk
+include $(XEN_ROOT)/config/Paths-tools.mk
 
 CONFIG_RUMP         := @CONFIG_RUMP@
 ifeq ($(CONFIG_RUMP),y)
diff --git a/configure.ac b/configure.ac
index 1843b52..fd5ebfd 100644
--- a/configure.ac
+++ b/configure.ac
@@ -5,10 +5,7 @@ AC_PREREQ([2.67])
 AC_INIT([Xen Hypervisor], m4_esyscmd([./version.sh ./xen/Makefile]),
     [xen-devel@lists.xen.org], [xen], [http://www.xen.org/])
 AC_CONFIG_SRCDIR([./xen/common/kernel.c])
-AC_CONFIG_FILES([
-	config/Toplevel.mk
-	config/Paths.mk
-])
+AC_CONFIG_FILES([config/Toplevel.mk])
 
 AC_CANONICAL_HOST
 
diff --git a/docs/Makefile b/docs/Makefile
index 854fb50..fc26158 100644
--- a/docs/Makefile
+++ b/docs/Makefile
@@ -92,7 +92,8 @@ clean:
 
 .PHONY: distclean
 distclean: clean
-	rm -rf $(XEN_ROOT)/config/Docs.mk config.log config.status config.cache \
+	rm -rf $(XEN_ROOT)/config/Docs.mk $(XEN_ROOT)/config/Paths-docs.mk \
+		config.log config.status config.cache \
 		autom4te.cache
 
 .PHONY: install-man-pages
diff --git a/docs/configure.ac b/docs/configure.ac
index bc77f49..24138b5 100644
--- a/docs/configure.ac
+++ b/docs/configure.ac
@@ -5,7 +5,10 @@ AC_PREREQ([2.67])
 AC_INIT([Xen Hypervisor Documentation], m4_esyscmd([../version.sh ../xen/Makefile]),
     [xen-devel@lists.xen.org], [xen], [http://www.xen.org/])
 AC_CONFIG_SRCDIR([misc/xen-command-line.markdown])
-AC_CONFIG_FILES([../config/Docs.mk])
+AC_CONFIG_FILES([
+../config/Docs.mk
+../config/Paths-docs.mk:../config/Paths.mk.in
+])
 AC_CONFIG_AUX_DIR([../])
 
 # M4 Macro includes
diff --git a/stubdom/Makefile b/stubdom/Makefile
index d9e7e40..5172464 100644
--- a/stubdom/Makefile
+++ b/stubdom/Makefile
@@ -564,7 +564,8 @@ downloadclean: patchclean
 
 .PHONY: distclean
 distclean: downloadclean
-	rm -rf config.log config.status config.cache autom4te.cache ../config/Stubdom.mk
+	rm -rf config.log config.status config.cache autom4te.cache \
+		../config/Stubdom.mk ../config/Paths-stubdom.mk
 
 ifeq (,$(findstring clean,$(MAKECMDGOALS)))
 $(XEN_ROOT)/config/Stubdom.mk:
diff --git a/stubdom/configure.ac b/stubdom/configure.ac
index 9fec853..31c3356 100644
--- a/stubdom/configure.ac
+++ b/stubdom/configure.ac
@@ -5,7 +5,10 @@ AC_PREREQ([2.67])
 AC_INIT([Xen Hypervisor Stub Domains], m4_esyscmd([../version.sh ../xen/Makefile]),
     [xen-devel@lists.xen.org], [xen], [http://www.xen.org/])
 AC_CONFIG_SRCDIR([xenstore-minios.cfg])
-AC_CONFIG_FILES([../config/Stubdom.mk])
+AC_CONFIG_FILES([
+../config/Stubdom.mk
+../config/Paths-stubdom.mk:../config/Paths.mk.in
+])
 AC_CONFIG_AUX_DIR([../])
 
 AC_CANONICAL_HOST
diff --git a/tools/Makefile b/tools/Makefile
index 966354a..e8c41b2 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -108,7 +108,8 @@ clean: subdirs-clean
 distclean: subdirs-distclean
 	rm -rf qemu-xen-traditional-dir qemu-xen-traditional-dir-remote
 	rm -rf qemu-xen-dir qemu-xen-dir-remote
-	rm -rf ../config/Tools.mk config.h config.log config.status \
+	rm -rf ../config/Tools.mk ../config/Paths-tools.mk \
+		config.h config.log config.status \
 		config.cache autom4te.cache
 
 ifneq ($(XEN_COMPILE_ARCH),$(XEN_TARGET_ARCH))
diff --git a/tools/configure.ac b/tools/configure.ac
index d31c2f3..aec2a31 100644
--- a/tools/configure.ac
+++ b/tools/configure.ac
@@ -7,6 +7,7 @@ AC_INIT([Xen Hypervisor Tools], m4_esyscmd([../version.sh ../xen/Makefile]),
 AC_CONFIG_SRCDIR([libxl/libxl.c])
 AC_CONFIG_FILES([
 ../config/Tools.mk
+../config/Paths-tools.mk:../config/Paths.mk.in
 hotplug/FreeBSD/rc.d/xencommons
 hotplug/Linux/init.d/sysconfig.xencommons
 hotplug/Linux/init.d/xen-watchdog
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] config: generate Paths-$SUBSYSTEM.mk
  2015-05-06  5:02 [PATCH v2] config: generate Paths-$SUBSYSTEM.mk Wei Liu
@ 2015-05-08 14:43 ` Ian Campbell
  2015-05-08 15:20   ` Wei Liu
  0 siblings, 1 reply; 3+ messages in thread
From: Ian Campbell @ 2015-05-08 14:43 UTC (permalink / raw)
  To: Wei Liu; +Cc: Ian Jackson, xen-devel

On Wed, 2015-05-06 at 06:02 +0100, Wei Liu wrote:
> ... so that we can run ./configure in each subsystem.
> 
> Make inclusion of Paths-*.mk mandatory. Skip generating a global
> Paths.mk since it's of no use anymore.
> 
> Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> Cc: Ian Campbell <ian.campbell@citrix.com>
> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> Acked-by: Ian Campbell <ian.campbell@citrix.com>
> ---
> v2: update .gitignore, delete Paths-*.mk in distclean
> 
> Please rerun autogen.sh when applying this patch.

With this applied the list of installed files changes in a way which is
clearly not desirable ;-).

Looks like the stubdom case is not correctly handled?

--- ../FILE_LIST.BASE.staging.x86_64    2015-04-15 16:03:29.000000000 +0100
+++ ../FILE_LIST.staging.x86_64 2015-05-08 15:41:21.000000000 +0100
@@ -56,6 +56,9 @@
 dist/install/etc/xen/xl.conf
 dist/install/etc/xen/xlexample.hvm
 dist/install/etc/xen/xlexample.pvlinux
+dist/install@LIBEXEC_BIN@
+dist/install@LIBEXEC_BIN@/stubdom-dm
+dist/install@LIBEXEC_BIN@/stubdompath.sh
 dist/install.sh
 dist/install/usr
 dist/install/usr/local
@@ -337,8 +340,6 @@
 dist/install/usr/local/lib/xen/bin/qemu-nbd
 dist/install/usr/local/lib/xen/bin/qemu-system-i386
 dist/install/usr/local/lib/xen/bin/readnotes
-dist/install/usr/local/lib/xen/bin/stubdom-dm
-dist/install/usr/local/lib/xen/bin/stubdompath.sh
 dist/install/usr/local/lib/xen/bin/xenconsole
 dist/install/usr/local/lib/xen/bin/xenctx
 dist/install/usr/local/lib/xen/bin/xendomains
@@ -347,12 +348,6 @@
 dist/install/usr/local/lib/xen/bin/xenpvnetboot
 dist/install/usr/local/lib/xen/boot
 dist/install/usr/local/lib/xen/boot/hvmloader
-dist/install/usr/local/lib/xen/boot/ioemu-stubdom.gz
-dist/install/usr/local/lib/xen/boot/pv-grub-x86_32.gz
-dist/install/usr/local/lib/xen/boot/pv-grub-x86_64.gz
-dist/install/usr/local/lib/xen/boot/vtpmmgr-stubdom.gz
-dist/install/usr/local/lib/xen/boot/vtpm-stubdom.gz
-dist/install/usr/local/lib/xen/boot/xenstore-stubdom.gz
 dist/install/usr/local/lib/xen/etc
 dist/install/usr/local/lib/xen/etc/qemu
 dist/install/usr/local/lib/xen/etc/qemu/target-x86_64.conf
@@ -785,4 +780,11 @@
 dist/install/var/log/xen
 dist/install/var/xen
 dist/install/var/xen/dump
+dist/install@XENFIRMWAREDIR@
+dist/install@XENFIRMWAREDIR@/ioemu-stubdom.gz
+dist/install@XENFIRMWAREDIR@/pv-grub-x86_32.gz
+dist/install@XENFIRMWAREDIR@/pv-grub-x86_64.gz
+dist/install@XENFIRMWAREDIR@/vtpmmgr-stubdom.gz
+dist/install@XENFIRMWAREDIR@/vtpm-stubdom.gz
+dist/install@XENFIRMWAREDIR@/xenstore-stubdom.gz
 dist/README
*** FILES DIFFER ***

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] config: generate Paths-$SUBSYSTEM.mk
  2015-05-08 14:43 ` Ian Campbell
@ 2015-05-08 15:20   ` Wei Liu
  0 siblings, 0 replies; 3+ messages in thread
From: Wei Liu @ 2015-05-08 15:20 UTC (permalink / raw)
  To: Ian Campbell; +Cc: Ian Jackson, Wei Liu, xen-devel

On Fri, May 08, 2015 at 03:43:14PM +0100, Ian Campbell wrote:
> On Wed, 2015-05-06 at 06:02 +0100, Wei Liu wrote:
> > ... so that we can run ./configure in each subsystem.
> > 
> > Make inclusion of Paths-*.mk mandatory. Skip generating a global
> > Paths.mk since it's of no use anymore.
> > 
> > Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> > Cc: Ian Campbell <ian.campbell@citrix.com>
> > Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> > Acked-by: Ian Campbell <ian.campbell@citrix.com>
> > ---
> > v2: update .gitignore, delete Paths-*.mk in distclean
> > 
> > Please rerun autogen.sh when applying this patch.
> 
> With this applied the list of installed files changes in a way which is
> clearly not desirable ;-).
> 
> Looks like the stubdom case is not correctly handled?
> 

Er... stubdom.

I will respin and double check.

Wei.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-05-08 15:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-06  5:02 [PATCH v2] config: generate Paths-$SUBSYSTEM.mk Wei Liu
2015-05-08 14:43 ` Ian Campbell
2015-05-08 15:20   ` Wei Liu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.