* [PATCH] kbuild: Get more focus on warnings
@ 2003-05-25 21:08 Sam Ravnborg
2003-05-25 21:22 ` Sam Ravnborg
0 siblings, 1 reply; 2+ messages in thread
From: Sam Ravnborg @ 2003-05-25 21:08 UTC (permalink / raw)
To: Linus Torvalds, linux-kernel
The following patch put focus on warnings when compiling.
Kbuild has for a long period supported a quit mode.
The quit mode prints a single line for each file compiled,
thus warnings becomes much more visible.
Example:
CC kernel/uid16.o
CC kernel/ksyms.o
kernel/ksyms.c:441: warning: `__check_region' is deprecated (declared at include/linux/ioport.h:113)
CC kernel/module.o
The old output format can obtained like this:
$ make V=1
or more permanent with the following assignment in .bashrc:
KBUILD_VERBOSE=1
If one is concerned about warnings the output from a kernel
build is redirected to a file, or make -s is used.
The advantage of this patch is that the default settings puts focus
on warnings while still providing the user with feedback about the
build process.
Kai Germaschewski has done most of the initial work enabling this
feature, originally inspired from Keith Owens kbuild-2.5 patchset.
Sam
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] kbuild: Get more focus on warnings
2003-05-25 21:08 [PATCH] kbuild: Get more focus on warnings Sam Ravnborg
@ 2003-05-25 21:22 ` Sam Ravnborg
0 siblings, 0 replies; 2+ messages in thread
From: Sam Ravnborg @ 2003-05-25 21:22 UTC (permalink / raw)
To: Linus Torvalds, linux-kernel
On Sun, May 25, 2003 at 11:08:39PM +0200, Sam Ravnborg wrote:
> The following patch put focus on warnings when compiling.
OK, here is the patch.
Sam
===== Makefile 1.406 vs edited =====
--- 1.406/Makefile Thu May 8 07:12:53 2003
+++ edited/Makefile Sun May 25 22:56:07 2003
@@ -107,7 +107,7 @@
# If it is set to "silent_", nothing wil be printed at all, since
# the variable $(silent_cmd_cc_o_c) doesn't exist.
-# For now, leave verbose as default
+# To put more focus on warnings, less verbose as default
ifdef V
ifeq ("$(origin V)", "command line")
@@ -115,7 +115,7 @@
endif
endif
ifndef KBUILD_VERBOSE
- KBUILD_VERBOSE = 1
+ KBUILD_VERBOSE = 0
endif
ifdef C
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2003-05-25 21:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-05-25 21:08 [PATCH] kbuild: Get more focus on warnings Sam Ravnborg
2003-05-25 21:22 ` Sam Ravnborg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®