From: Felix Zielcke <fzielcke@z-51.de>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: [PATH] grub-mkrelpath
Date: Wed, 04 Nov 2009 12:07:29 +0100 [thread overview]
Message-ID: <1257332849.5000.54.camel@fz.local> (raw)
In-Reply-To: <bf230c153fb88afda22753b206d0e45a@scarlet.be>
Am Mittwoch, den 04.11.2009, 10:55 +0100 schrieb rubisher:
> > Right trunk contains my -I($srcdir)/include fix.
> > merged now.
> Sorry I don't agree with your fix:
As you might have thought already from the following sentence, I already
suspected that it's not the proper way.
By the way this topic really belongs to the -nostdinc Thread from
Robert.
This is not at all specific to my grub-mkrelpath.
At least this fixed building with a seperate build directory.
>
>
> > But I still have the feeling using -isystem is wrong at all or at least
> > the =.
> Well as far as I understand, yes for cross-compiling that would make stuff
> easier?
>
> A quick look at linux kernel build, it shows that also use -isystem but
> not yet with '=' option so may be my previous patch would be more relevant.
>
> I let you appreciate.
IMO we should use the same approach as Linux kernel does. Using -isystem
just for the gcc include headers.
I don't think gcc should treat our own headers as system headers.
> Tx,
> J.
>
> PS: any idea where (in which part of this proj) should I have a look to
> learn why ofs doesn't reach to find disks and their structures?)
>
No clue.
Better ask such questions in a seperate mail, not hidden in a thread
which has nothing to do with this. The chances are much higer that then
who can answer the qestion will read it.
--
Felix Zielcke
Proud Debian Maintainer and GNU GRUB developer
next prev parent reply other threads:[~2009-11-04 11:07 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-28 12:53 [PATH] grub-mkrelpath Felix Zielcke
2009-08-28 16:28 ` Robert Millan
2009-08-28 17:58 ` Felix Zielcke
2009-08-28 23:55 ` Robert Millan
2009-08-29 0:58 ` Vladimir 'phcoder' Serbinenko
2009-08-29 7:51 ` Felix Zielcke
2009-11-01 15:39 ` Felix Zielcke
2009-11-01 22:04 ` Robert Millan
2009-11-02 8:54 ` Felix Zielcke
2009-11-02 13:45 ` Robert Millan
2009-11-02 15:41 ` Felix Zielcke
2009-11-03 10:21 ` rubisher
2009-11-03 10:33 ` Felix Zielcke
2009-11-03 13:51 ` rubisher
2009-11-03 14:09 ` Felix Zielcke
2009-11-04 9:55 ` rubisher
2009-11-04 11:07 ` Felix Zielcke [this message]
2009-11-04 14:59 ` rubisher
2009-11-04 15:55 ` Felix Zielcke
2009-11-11 22:43 ` Felix Zielcke
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1257332849.5000.54.camel@fz.local \
--to=fzielcke@z-51.de \
--cc=grub-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.