[PATCH] add -I$(KERNEL_HEADERS) in test/Rules.mak

Bernhard Reutner-Fischer rep.dot.nop at gmail.com
Wed Jul 22 07:25:22 UTC 2009


On Wed, Jul 22, 2009 at 03:14:59AM +0200, Denys Vlasenko wrote:
>On Tuesday 21 July 2009 19:24, Bernhard Reutner-Fischer wrote:
>> On Tue, Jul 21, 2009 at 07:07:54PM +0200, Denys Vlasenko wrote:
>> 
>> >> In my POV this is a configuration issue and not libc's business.
>> >
>> >Then why do we have KERNEL_HEADERS?
>> 
>> So add the hunks that CFLAGS+=-I$(KERNEL_HEADERS) from Rules.mk to
>> the shadow-copy in test/
>
>Is this ok with you, Bernhard?

sure, ok.
>
>
>diff -d -urpN uClibc.7/test/Rules.mak uClibc.8/test/Rules.mak
>--- uClibc.7/test/Rules.mak	2009-07-20 21:53:29.000000000 +0200
>+++ uClibc.8/test/Rules.mak	2009-07-22 03:09:09.000000000 +0200
>@@ -86,6 +86,10 @@ CC_IPREFIX := $(shell $(CC) --print-file
> CC_INC := -I$(dir $(CC_IPREFIX))include-fixed -I$(CC_IPREFIX)
> CFLAGS += $(CC_INC)
> 
>+ifneq ($(KERNEL_HEADERS),)
>+CFLAGS += -I$(top_builddir)$(KERNEL_HEADERS)
>+endif
>+
> # Can't add $(OPTIMIZATION) here, it may be target-specific.
> # Just adding -Os for now.
> HOST_CFLAGS    += $(XWARNINGS) -Os $(XCOMMON_CFLAGS)
>
>
>
>
>
>with this patch, I was able to run a few tests.
>Then it broke in test/librt/*:

my fault, please install this one if it helps:

index 362d5d5..ca60d72 100644
--- a/test/librt/Makefile
+++ b/test/librt/Makefile
@@ -1,6 +1,9 @@
 # uClibc shm tests
 # Licensed under the LGPL v2.1, see the file COPYING.LIB in this tarball.
 
+top_builddir=../../
+include ../Rules.mak
+-include Makefile.in
 include ../Test.mak
 
 LDFLAGS_shmtest := -lrt


TIA


More information about the uClibc mailing list