* [PATCH] Makefile include path ordering
@ 2005-05-16 21:02 Rik van Riel
2005-05-17 18:36 ` Sam Ravnborg
0 siblings, 1 reply; 2+ messages in thread
From: Rik van Riel @ 2005-05-16 21:02 UTC (permalink / raw)
To: linux-kernel; +Cc: xen-devel
The arch Makefile may override the include path order, which is
used by Xen (and UML?) to make sure include/asm-xen is searched
before include/asm-i386.
The Makefile change to 2.6.12-rc4 made the top Makefile always
override the value specified by the arch Makefile. This trivial
patch makes the Xen kernel compile again.
Signed-off-by: Rik van Riel <riel@redhat.com>
--- linux-2.6.11/Makefile.order 2005-05-16 16:20:20.000000000 -0400
+++ linux-2.6.11/Makefile 2005-05-16 16:21:30.000000000 -0400
@@ -530,7 +530,7 @@
include $(srctree)/arch/$(ARCH)/Makefile
# arch Makefile may override CC so keep this after arch Makefile is included
-NOSTDINC_FLAGS := -nostdinc -isystem $(shell $(CC) -print-file-name=include)
+NOSTDINC_FLAGS += -nostdinc -isystem $(shell $(CC) -print-file-name=include)
CHECKFLAGS += $(NOSTDINC_FLAGS)
# warn about C99 declaration after statement
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Makefile include path ordering
2005-05-16 21:02 [PATCH] Makefile include path ordering Rik van Riel
@ 2005-05-17 18:36 ` Sam Ravnborg
0 siblings, 0 replies; 2+ messages in thread
From: Sam Ravnborg @ 2005-05-17 18:36 UTC (permalink / raw)
To: Rik van Riel; +Cc: linux-kernel, xen-devel
On Mon, May 16, 2005 at 05:02:17PM -0400, Rik van Riel wrote:
> The arch Makefile may override the include path order, which is
> used by Xen (and UML?) to make sure include/asm-xen is searched
> before include/asm-i386.
gcc will respect the order of -I options assinged to CFLAGS
So adding xen support at the proper (topmost) place in
arch/i386/Makefile should satisfy this requirement.
Also playing with NOSTDINC_FLAGS will most likely break builds with O=
since kbuild does not mangle patchs specified with NOSTDINC
Sam
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2005-05-17 18:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-05-16 21:02 [PATCH] Makefile include path ordering Rik van Riel
2005-05-17 18:36 ` 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®