[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add -nostdinc to TARGET_CFLAGS
From: |
Felix Zielcke |
Subject: |
Re: [PATCH] Add -nostdinc to TARGET_CFLAGS |
Date: |
Sun, 15 Nov 2009 13:42:56 +0100 |
Am Sonntag, den 15.11.2009, 13:32 +0100 schrieb Robert Millan:
> On Sun, Nov 15, 2009 at 12:50:39PM +0100, Felix Zielcke wrote:
> > > > $ srcdir=$PWD builddir=$PWD gcc -isystem=$srcdir/include
> > > > -I$srcdir/include -I$builddir -I$builddir/include test.c -o test && ls
> > > > test
> > > > test
> > > > $ srcdir=$PWD builddir=$PWD gcc -nostdinc -isystem $(gcc
> > > > -print-file-name=include) -I$srcdir/include -I$builddir
> > > > -I$builddir/include test.c
> > > > test.c:2:20: error: stdint.h: No such file or directory
> > >
> > > We used -isystem as a way of excluding system headers but not gcc headers.
> > > With the -print-file-name trick this seems to be no longer necessary,
> > > right?
> > >
> > > So why not "-nostdinc -I$(gcc -print-file-name=include)" instead?
> >
> > I think we should use -isystem for the gcc internal header files not -I,
> > but I just tested experimental branch with -I instead of -isystem and
> > also compiles cleanly (except the usual grub.texi warnings)
>
> Seems fine. Thanks for investigating this.
You're welcome.
I commited this now except that I used now $TARGET_CC.
Makes more sense inside TARGET_CPPFLAGS.
--
Felix Zielcke
Proud Debian Maintainer and GNU GRUB developer
- Re: [PATCH] Add -nostdinc to TARGET_CFLAGS, Felix Zielcke, 2009/11/04
- Re: [PATCH] Add -nostdinc to TARGET_CFLAGS, Felix Zielcke, 2009/11/14
- Re: [PATCH] Add -nostdinc to TARGET_CFLAGS, Robert Millan, 2009/11/15
- Re: [PATCH] Add -nostdinc to TARGET_CFLAGS, Felix Zielcke, 2009/11/15
- Re: [PATCH] Add -nostdinc to TARGET_CFLAGS, Robert Millan, 2009/11/15
- Re: [PATCH] Add -nostdinc to TARGET_CFLAGS, Felix Zielcke, 2009/11/15
- Re: [PATCH] Add -nostdinc to TARGET_CFLAGS, Robert Millan, 2009/11/15
- Re: [PATCH] Add -nostdinc to TARGET_CFLAGS,
Felix Zielcke <=