linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Warn about config.h inclusion
@ 2006-08-23 23:19 Alexey Dobriyan
  2006-08-24  2:01 ` Sam Ravnborg
  0 siblings, 1 reply; 2+ messages in thread
From: Alexey Dobriyan @ 2006-08-23 23:19 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linux-kernel

Otherwise feature adders won't feel it's unneeded..

Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
---

 include/linux/config.h |    1 +
 1 file changed, 1 insertion(+)

--- a/include/linux/config.h
+++ b/include/linux/config.h
@@ -3,6 +3,7 @@ #define _LINUX_CONFIG_H
 /* This file is no longer in use and kept only for backward compatibility.
  * autoconf.h is now included via -imacros on the commandline
  */
+#warning Don't include me. I'm already in via -imacros. I'll dissappear. You have been warned.
 #include <linux/autoconf.h>
 
 #endif


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Warn about config.h inclusion
  2006-08-23 23:19 [PATCH] Warn about config.h inclusion Alexey Dobriyan
@ 2006-08-24  2:01 ` Sam Ravnborg
  0 siblings, 0 replies; 2+ messages in thread
From: Sam Ravnborg @ 2006-08-24  2:01 UTC (permalink / raw)
  To: Alexey Dobriyan; +Cc: Andrew Morton, linux-kernel

On Thu, Aug 24, 2006 at 03:19:02AM +0400, Alexey Dobriyan wrote:
> Otherwise feature adders won't feel it's unneeded..
> 
> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
> ---
> 
>  include/linux/config.h |    1 +
>  1 file changed, 1 insertion(+)
> 
> --- a/include/linux/config.h
> +++ b/include/linux/config.h
> @@ -3,6 +3,7 @@ #define _LINUX_CONFIG_H
>  /* This file is no longer in use and kept only for backward compatibility.
>   * autoconf.h is now included via -imacros on the commandline
s/imacros/include/

Try make V=1 to see it.
It was -imacros in the early beginning but due to troubles with some binutils version it was changed IIRC.

     Sam

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2006-08-24  1:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-08-23 23:19 [PATCH] Warn about config.h inclusion Alexey Dobriyan
2006-08-24  2:01 ` Sam Ravnborg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).