diff options
author | Keir Fraser <keir.fraser@citrix.com> | 2009-01-12 10:16:05 +0000 |
---|---|---|
committer | Keir Fraser <keir.fraser@citrix.com> | 2009-01-12 10:16:05 +0000 |
commit | 9f8501582f1c60c0a631a89499fb81d8f197fb30 (patch) | |
tree | 9f7b768cc4205e09038c454d339b871099a7c324 /tools/console | |
parent | ceff4bb8a36948e4216022511b263831e7e7190f (diff) | |
download | xen-9f8501582f1c60c0a631a89499fb81d8f197fb30.tar.gz xen-9f8501582f1c60c0a631a89499fb81d8f197fb30.tar.bz2 xen-9f8501582f1c60c0a631a89499fb81d8f197fb30.zip |
Use -MMD -MF in tools/* rather than -Wp,-M...
If you use -MMD -MF then the correct .o filename is written to the
.*.d file as the compiler driver arranges everything. This was done
in 19010:275abe1c5d24 for the hypervisor.
In this patch we do the same elsewhere in the xen-unstable tree,
particularly tools/. Specifically:
* Change tools/Rules.mk to add -MMD -MF ... to CFLAGS and set DEPS.
* Remove -Wp,-MD... from every other Makefile
* Remove setting of DEPS from every other Makefile
* Ensure that every Makefile says -include $(DEPS)
* Ensure that every Makefile's clean target removes $(DEPS)
Some Makefiles were already halfway there, but often for a different
variable name eg PROG_DEP. The variable name is now standardised in
Rules.mk as DEPS.
I have done a test build with this change, on Debian etch.
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
Diffstat (limited to 'tools/console')
-rw-r--r-- | tools/console/Makefile | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/console/Makefile b/tools/console/Makefile index 10e909a0b6..7339e9e064 100644 --- a/tools/console/Makefile +++ b/tools/console/Makefile @@ -16,7 +16,7 @@ all: $(BIN) .PHONY: clean clean: - $(RM) *.a *.so *.o *.rpm $(BIN) + $(RM) *.a *.so *.o *.rpm $(BIN) $(DEPS) $(RM) client/*.o daemon/*.o xenconsoled: $(patsubst %.c,%.o,$(wildcard daemon/*.c)) @@ -33,3 +33,5 @@ install: $(BIN) $(INSTALL_PROG) xenconsoled $(DESTDIR)/$(SBINDIR) $(INSTALL_DIR) $(DESTDIR)$(PRIVATE_BINDIR) $(INSTALL_PROG) xenconsole $(DESTDIR)$(PRIVATE_BINDIR) + +-include $(DEPS) |