All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Yoshinori K. Okuji" <okuji@enbug.org>
To: grub-devel@gnu.org
Subject: Re: multiboot
Date: Thu, 9 Dec 2004 03:16:04 +0100	[thread overview]
Message-ID: <200412090316.04926.okuji@enbug.org> (raw)
In-Reply-To: <87brd4q5wq.fsf@night.trouble.net>

On Wednesday 08 December 2004 14:24, Johan Rydberg wrote:
> The initial problem was that AC_CHECK_SIZEOF doesn't work if you're
> cross-compiling.  I fixed this by using AC_COMPILE_CHECK_SIZEOF [1]
> instead.

Ok.

> But the major problem is that parts of GRUB2 requires a C library to
> be installed, at least headers such as alloca.h and stdint.h.  Is
> this really a valid demand?

Why do we use stdint.h? As far as I see, it is used only for 
powerpc-specific code. This should be removed. GRUB must use its own 
types defined in include/grub/types.h.

alloca should not be used. It must be replaced with something else (e.g. 
grub_malloc). Note that alloca requires a C library in some 
architectures.

> Another issue regarding BUILD_CC.  The normal convention is that if
> you want to build something that should run on the build machine, you
> use something similar to BUILD_CC.  That's why I think the following
> patch is self-explaining.

You are right. I didn't notice the bug, because I usually specify CC and 
BUILD_CC manually.

> That brings us the next issue; BUILD_CC is used to build
> _everything_, including the host-specific object files, and since
> BUILD_CC is set to /usr/bin/gcc in my cross-compiler environment it
> will try to compile PPC assembler sources with a i386 compiler.  Not
> good.

Why? Host-specific objects are compiled by CC instead of BUILD_CC.

Okuji



  parent reply	other threads:[~2004-12-09  2:24 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-12-07 11:09 multiboot Johan Rydberg
2004-12-07 14:51 ` multiboot Marco Gerards
2004-12-08 10:44   ` multiboot Johan Rydberg
2004-12-08 11:59     ` multiboot Marco Gerards
2004-12-08 12:27       ` multiboot Johan Rydberg
2004-12-08 12:23         ` multiboot Marco Gerards
2004-12-08 13:24           ` multiboot Johan Rydberg
2004-12-08 13:34             ` multiboot Johan Rydberg
2004-12-08 14:57             ` multiboot Marco Gerards
2004-12-08 15:32               ` multiboot Johan Rydberg
2004-12-08 19:10                 ` multiboot Marco Gerards
2004-12-09  2:16             ` Yoshinori K. Okuji [this message]
2004-12-09 11:42               ` multiboot Johan Rydberg
2004-12-09 13:51                 ` multiboot Marco Gerards

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=200412090316.04926.okuji@enbug.org \
    --to=okuji@enbug.org \
    --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.