Cleaner Makefiles for VM-Tools

PATCH New aq 1 participant 1 message Apr 12, 2005 Kbuild

lore

Share & tools

lore

aq

author

2005-04-12 07:26 UTC #0 lore raw

PATCH 1/1 Cleaner Makefiles for VM-Tools

hi Anthony and VM-Tools team,

Here is the small patch for VM-Tools.

List of changes:
- cleaner Makefile(s)
- Remove some unecessary targets (thanks to default Makefile rules)
- Addition of uninstall target: now users can uninstall vm-tools.

$diffstat vm-tools.diff.txt

 Makefile          |    6 ++++++
 docs/Makefile     |    5 +++++
 docs/man/Makefile |   21 +++++++++------------
 scripts/Makefile  |    3 +++
 src/Makefile      |   30 ++++++++++--------------------
 xen/Makefile      |    4 ++++
 6 files changed, 37 insertions(+), 32 deletions(-)

Signed-of-by Nguyen Anh Quynh <[email protected]>

Best regards,
aq

docs/Makefile +5 -0

diff -Nurp vm-tools-0.0.4/docs/Makefile vm-tools-0.0.4-aq/docs/Makefile
@@ -21,8 +21,13 @@
 ###############################################################################
 
 all:
+	true
 
 clean:
+	true
 
 install:
 	$(MAKE) -C man install
+
+uninstall:
+	$(MAKE) -C man uninstall

docs/man/Makefile +9 ~~-12~~

diff -Nurp vm-tools-0.0.4/docs/man/Makefile vm-tools-0.0.4-aq/docs/man/Makefile
@@ -23,21 +23,18 @@
 INSTALL=install -D
 PREFIX=/usr/share
 GZIP=gzip -f
+GZIPEXT=gz
+MANS=vm-balloon.1 vm-create.1 vm-list.1 vm-console.1 vm-shutdown.1 vm.1
 
 all:
+	true
 
 clean:
+	true
 
 install:
-	$(INSTALL) vm-balloon.1 $(DESTDIR)$(PREFIX)/man/man1/vm-balloon.1 && \
-	$(GZIP) $(DESTDIR)$(PREFIX)/man/man1/vm-balloon.1
-	$(INSTALL) vm-create.1 $(DESTDIR)$(PREFIX)/man/man1/vm-create.1 && \
-	$(GZIP) $(DESTDIR)$(PREFIX)/man/man1/vm-create.1
-	$(INSTALL) vm-list.1 $(DESTDIR)$(PREFIX)/man/man1/vm-list.1 && \
-	$(GZIP) $(DESTDIR)$(PREFIX)/man/man1/vm-list.1
-	$(INSTALL) vm-console.1 $(DESTDIR)$(PREFIX)/man/man1/vm-console.1 && \
-	$(GZIP) $(DESTDIR)$(PREFIX)/man/man1/vm-console.1
-	$(INSTALL) vm-shutdown.1 $(DESTDIR)$(PREFIX)/man/man1/vm-shutdown.1 &&\
-	$(GZIP) $(DESTDIR)$(PREFIX)/man/man1/vm-shutdown.1
-	$(INSTALL) vm.1 $(DESTDIR)$(PREFIX)/man/man1/vm.1 &&\
-	$(GZIP) $(DESTDIR)$(PREFIX)/man/man1/vm.1
+	$(foreach i,$(MANS),$(INSTALL) $i $(DESTDIR)$(PREFIX)/man/man1/$i && \
+						$(GZIP) $(DESTDIR)$(PREFIX)/man/man1/$i;)
+
+uninstall:
+	$(foreach i,$(MANS),$(RM) $(DESTDIR)$(PREFIX)/man/man1/$i.$(GZIPEXT);)

Makefile +6 -0

diff -Nurp vm-tools-0.0.4/Makefile vm-tools-0.0.4-aq/Makefile
@@ -33,3 +33,9 @@ install:
 	$(MAKE) -C docs install && \
 	$(MAKE) -C scripts install && \
 	$(MAKE) -C src install
+
+uninstall:
+	$(MAKE) -C xen uninstall && \
+	$(MAKE) -C docs uninstall && \
+	$(MAKE) -C scripts uninstall && \
+	$(MAKE) -C src uninstall

scripts/Makefile +3 -0

diff -Nurp vm-tools-0.0.4/scripts/Makefile vm-tools-0.0.4-aq/scripts/Makefile
@@ -29,5 +29,8 @@ all:
 install: 	all
 	$(INSTALL) -m755 vm $(DESTDIR)$(PREFIX)/bin/vm
 
+uninstall: 	all
+	$(RM) $(DESTDIR)$(PREFIX)/bin/vm
+
 clean:
 	true

src/Makefile +10 ~~-20~~

diff -Nurp vm-tools-0.0.4/src/Makefile vm-tools-0.0.4-aq/src/Makefile
@@ -27,28 +27,18 @@ SRCDIR=..
 INCDIR=$(SRCDIR)/include
 CFLAGS=-g -Wshadow -Wall -I$(INCDIR) 
 LDLIBS=-L$(SRCDIR)/xen -lxen -lxc
+PROGS=vm-balloon vm-console vm-create vm-list vm-maxmem vm-pause vm-sched vm-shutdown vm-top
 
-all: vm-balloon vm-console vm-create vm-list vm-maxmem vm-pause vm-sched vm-shutdown vm-top
+all: $(PROGS)
 
-vm-top: vm-top.o
-vm-balloon: vm-balloon.o
-vm-console: vm-console.o
-vm-create: vm-create.o netif.o blkif.o console.o vdev.o memmap.o table.o
-vm-list: vm-list.o
-vm-maxmem: vm-maxmem.o
-vm-pause: vm-pause.o
-vm-sched: vm-sched.o
-vm-shutdown: vm-shutdown.o
+vm-create: netif.o blkif.o console.o vdev.o memmap.o table.o
 
 install: all
-	$(INSTALL) -m755 vm-balloon $(DESTDIR)$(PREFIX)/bin/vm-balloon
-	$(INSTALL) -m755 vm-console $(DESTDIR)$(PREFIX)/bin/vm-console
-	$(INSTALL) -m755 vm-create $(DESTDIR)$(PREFIX)/bin/vm-create
-	$(INSTALL) -m755 vm-list $(DESTDIR)$(PREFIX)/bin/vm-list
-	$(INSTALL) -m755 vm-maxmem $(DESTDIR)$(PREFIX)/bin/vm-maxmem
-	$(INSTALL) -m755 vm-pause $(DESTDIR)$(PREFIX)/bin/vm-pause
-	$(INSTALL) -m755 vm-sched  $(DESTDIR)$(PREFIX)/bin/vm-sched
-	$(INSTALL) -m755 vm-shutdown $(DESTDIR)$(PREFIX)/bin/vm-shutdown
-	$(INSTALL) -m755 vm-top $(DESTDIR)$(PREFIX)/bin/vm-top
+	$(foreach i,$(PROGS),$(INSTALL) -m755 $i $(DESTDIR)$(PREFIX)/bin/$i;)
 
-clean:; $(RM) *.o vm-balloon vm-console vm-create vm-list vm-maxmem vm-pause vm-sched vm-shutdown vm-top
+uninstall: 
+	$(foreach i,$(PROGS),$(RM) $(RM) $(DESTDIR)$(PREFIX)/bin/$i;)
+
+clean:
+	$(RM) *.o 
+	$(RM) $(PROGS)

xen/Makefile +4 -0

diff -Nurp vm-tools-0.0.4/xen/Makefile vm-tools-0.0.4-aq/xen/Makefile
@@ -37,4 +37,8 @@ install: libxen.a
                    $(DESTDIR)$(PREFIX)/include/xen/ctrl_chan.h
 	$(INSTALL) -m644 libxen.a $(DESTDIR)$(PREFIX)/lib/libxen.a
 
+uninstall:
+	$(RM) $(DESTDIR)$(PREFIX)/include/xen/ctrl_chan.h
+	$(RM) $(DESTDIR)$(PREFIX)/lib/libxen.a
+
 clean:; $(RM) *.o libxen.a

_______________________________________________
Xen-devel mailing list
[email protected]
http://lists.xensource.com/xen-devel