mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [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®