[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen master] build: move debug{, _symbols} to tools/Rules.mk
commit df6f51347318359575f972bbee9e966beaa85874 Author: Wei Liu <wei.liu2@xxxxxxxxxx> AuthorDate: Fri Dec 23 12:24:16 2016 +0000 Commit: Wei Liu <wei.liu2@xxxxxxxxxx> CommitDate: Thu Jan 5 16:36:07 2017 +0000 build: move debug{,_symbols} to tools/Rules.mk 31d41d7b tried to make debug affect tools build only but failed to take care of debug_symbols (which appends "-g" to CFLAGS). Move both to tools/Rules.mk at once in this patch. Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx> Acked-by: Jan Beulich <jbeulich@xxxxxxxx> --- Config.mk | 9 --------- tools/Rules.mk | 8 ++++++++ 2 files changed, 8 insertions(+), 9 deletions(-) diff --git a/Config.mk b/Config.mk index b26e15c..189a443 100644 --- a/Config.mk +++ b/Config.mk @@ -16,11 +16,6 @@ or = $(if $(strip $(1)),$(1),$(if $(strip $(2)),$(2),$(if $(strip $(3)),$( -include $(XEN_ROOT)/.config -# A debug build of tools? -# Hypervisor debug build is controlled by Kconfig. -debug ?= y -debug_symbols ?= $(debug) - XEN_COMPILE_ARCH ?= $(shell uname -m | sed -e s/i.86/x86_32/ \ -e s/i86pc/x86_32/ -e s/amd64/x86_64/ \ -e s/armv7.*/arm32/ -e s/armv8.*/arm64/ \ @@ -211,10 +206,6 @@ define buildmakevars2header-closure $(call move-if-changed,$(1).tmp,$(1)) endef -ifeq ($(debug_symbols),y) -CFLAGS += -g -endif - CFLAGS += -fno-strict-aliasing CFLAGS += -std=gnu99 diff --git a/tools/Rules.mk b/tools/Rules.mk index 0e73690..9a87f18 100644 --- a/tools/Rules.mk +++ b/tools/Rules.mk @@ -26,6 +26,14 @@ CFLAGS_xeninclude = -I$(XEN_INCLUDE) XENSTORE_XENSTORED ?= y +# A debug build of tools? +debug ?= y +debug_symbols ?= $(debug) + +ifeq ($(debug_symbols),y) +CFLAGS += -g +endif + ifneq ($(nosharedlibs),y) INSTALL_SHLIB = $(INSTALL_PROG) SYMLINK_SHLIB = ln -sf -- generated by git-patchbot for /home/xen/git/xen.git#master _______________________________________________ Xen-changelog mailing list Xen-changelog@xxxxxxxxxxxxx https://lists.xenproject.org/xen-changelog
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |