From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MMRYX-0004dZ-0y for qemu-devel@nongnu.org; Thu, 02 Jul 2009 15:02:49 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MMRYS-0004bi-Dz for qemu-devel@nongnu.org; Thu, 02 Jul 2009 15:02:48 -0400 Received: from [199.232.76.173] (port=40726 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MMRYR-0004bb-S0 for qemu-devel@nongnu.org; Thu, 02 Jul 2009 15:02:43 -0400 Received: from rv-out-0708.google.com ([209.85.198.250]:35568) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MMRYR-0004iI-AT for qemu-devel@nongnu.org; Thu, 02 Jul 2009 15:02:43 -0400 Received: by rv-out-0708.google.com with SMTP id b17so570466rvf.22 for ; Thu, 02 Jul 2009 12:02:42 -0700 (PDT) Message-ID: <4A4D044D.7020003@codemonkey.ws> Date: Thu, 02 Jul 2009 14:02:37 -0500 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH] Replace signrom with shell script v3 References: <1246478901-6623-1-git-send-email-agraf@suse.de> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Blue Swirl Cc: Alexander Graf , qemu-devel@nongnu.org Blue Swirl wrote: > On 7/2/09, Blue Swirl wrote: > >> On 7/1/09, Alexander Graf wrote: >> > In order to not execute code we just compiled, let's replace signrom >> > with a shell script that does the same thing while staying compatible >> > to pretty much every system available. >> > >> > This should make cross-compilation for windows easier. >> >> >> Why don't you use $(HOST_CC) to compile the signrom binary? >> >> > > Like with this patch? Fixes Mingw32 compile for me. > If you're cross compiling, HOST_CC is the wrong compiler to use for the ROM build. For instance, if you were on a PPC host building an i386 QEMU package. Building signrom with HOST_CC may be reasonable. Regards, Anthony Liguori