aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSam Ravnborg <sam@ravnborg.org>2007-11-04 13:00:46 -0500
committerSam Ravnborg <sam@ravnborg.org>2007-11-04 13:00:46 -0500
commit69ee0b3522428a07ff1765446d631ecc7da6ae0f (patch)
treefe13c78e9f7e7fa528a7bd1bd72e387373668212
parentb4f555081fdd27d13e6ff39d455d5aefae9d2c0c (diff)
kbuild: do not pick up CFLAGS from the environment
Too many people have CFLAGS set to support building userspace. And now Kbuild picks up CFLAGS this caused troubles. Although people should realise that setting CFLAGS has a 'global' effect the impact on the kernel build is a suprise. So change kbuild to pick up value from KCFLAGS that is much less used. When kbuild pick up a value it will warn like this: Makefile:544: "WARNING: Appending $KCFLAGS (-O3) from environment to kernel $CFLAGS" Signed-off-by: Sam Ravnborg <sam@ravnborg.org> Cc: Thomas Bächler <thomas@archlinux.org> Cc: David Miller <davem@davemloft.net> Cc: Ingo Molnar <mingo@redhat.com>
-rw-r--r--Makefile19
1 files changed, 16 insertions, 3 deletions
diff --git a/Makefile b/Makefile
index 188c3b6b3fa9..cb740d4d4e8c 100644
--- a/Makefile
+++ b/Makefile
@@ -528,9 +528,22 @@ KBUILD_CFLAGS += $(call cc-option,-Wdeclaration-after-statement,)
528KBUILD_CFLAGS += $(call cc-option,-Wno-pointer-sign,) 528KBUILD_CFLAGS += $(call cc-option,-Wno-pointer-sign,)
529 529
530# Add user supplied CPPFLAGS, AFLAGS and CFLAGS as the last assignments 530# Add user supplied CPPFLAGS, AFLAGS and CFLAGS as the last assignments
531KBUILD_CPPFLAGS += $(CPPFLAGS) 531# But warn user when we do so
532KBUILD_AFLAGS += $(AFLAGS) 532warn-assign = \
533KBUILD_CFLAGS += $(CFLAGS) 533$(warning "WARNING: Appending $$K$(1) ($(K$(1))) from $(origin K$(1)) to kernel $$$(1)")
534
535ifneq ($(KCPPFLAGS),)
536 $(call warn-assign,CPPFLAGS)
537 KBUILD_CPPFLAGS += $(KCPPFLAGS)
538endif
539ifneq ($(KAFLAGS),)
540 $(call warn-assign,AFLAGS)
541 KBUILD_AFLAGS += $(KAFLAGS)
542endif
543ifneq ($(KCFLAGS),)
544 $(call warn-assign,CFLAGS)
545 KBUILD_CFLAGS += $(KCFLAGS)
546endif
534 547
535# Use --build-id when available. 548# Use --build-id when available.
536LDFLAGS_BUILD_ID = $(patsubst -Wl$(comma)%,%,\ 549LDFLAGS_BUILD_ID = $(patsubst -Wl$(comma)%,%,\