All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marco Gerards <metgerards@student.han.nl>
To: Johan Rydberg <jrydberg@night.trouble.net>
Cc: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: multiboot
Date: Wed, 08 Dec 2004 14:57:43 +0000	[thread overview]
Message-ID: <878y887s7s.fsf@marco.marco-g.com> (raw)
In-Reply-To: <87brd4q5wq.fsf@night.trouble.net> (Johan Rydberg's message of "Wed, 08 Dec 2004 14:24:37 +0100")

Johan Rydberg <jrydberg@night.trouble.net> writes:

> 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.

This sounds right to me, but I do not see this in the patch...

> 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?  

Is this required for grubof?  IIRC those headers are not a part of the
C library, but a part of gcc or libgcc or so.  Am I right about that?

Instead of alloca we can also use something like:

...

{
 int foo[size]
 ...
}

In most cases that will remove the dependency on alloca.  IMHO it is
better not to use alloca.

> 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.

That sounds sane to me.  I assume this fixes Timothy's problem as
well.

> 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.
>
> My suggestion is to create a new class for programs that should be
> host-specific, and let Utilities be compiled on the build machine.
> See the patch.  

Ok, that sounds sane.  But what if you really want to cross-compile a
complete system (including libc, GRUB, etc)?

Can you make one big patch with a changelog entry so it can be
committed after Okuji agrees with the patch?

Thanks,
Marco




  parent reply	other threads:[~2004-12-08 15:12 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             ` Marco Gerards [this message]
2004-12-08 15:32               ` multiboot Johan Rydberg
2004-12-08 19:10                 ` multiboot Marco Gerards
2004-12-09  2:16             ` multiboot Yoshinori K. Okuji
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=878y887s7s.fsf@marco.marco-g.com \
    --to=metgerards@student.han.nl \
    --cc=grub-devel@gnu.org \
    --cc=jrydberg@night.trouble.net \
    /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.