From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1HTIDy-0006jT-AK for qemu-devel@nongnu.org; Mon, 19 Mar 2007 09:48:34 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1HTIDv-0006j5-VL for Qemu-devel@nongnu.org; Mon, 19 Mar 2007 09:48:33 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1HTIDv-0006j2-PB for Qemu-devel@nongnu.org; Mon, 19 Mar 2007 08:48:31 -0500 Received: from phoenix.bawue.net ([193.7.176.60] helo=mail.bawue.net) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1HTICS-0003sl-VF for Qemu-devel@nongnu.org; Mon, 19 Mar 2007 09:47:01 -0400 Date: Mon, 19 Mar 2007 13:47:15 +0000 Subject: Re: [Qemu-devel] PATCH: arm, mips and sparc targets on Host Solaris/sparc Message-ID: <20070319134715.GE7744@networkno.de> References: <7369890.1174311061508.JavaMail.root@eastrmwml08.cox.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <7369890.1174311061508.JavaMail.root@eastrmwml08.cox.net> From: Thiemo Seufer Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Ben Taylor Cc: Qemu-devel@nongnu.org Ben Taylor wrote: > > This patch is to fix (correctly) running mips, sparc and arm targets > on a Solaris/Sparc host. > > This patch reverts the patch posted February 2nd, 2007, (found at > http://lists.gnu.org/archive/html/qemu-devel/2007-02/msg00008.html > for reference). Juergen Keil found that config.h was not included in > target-arm/exec.h and config.h was included *after* dyngen-exec.h > in target-sparc/exec.h. [snip] > diff -ruN qemu-ORIG/target-sparc/exec.h qemu/target-sparc/exec.h > --- qemu-ORIG/target-sparc/exec.h 2007-02-01 20:03:34.000000000 -0500 > +++ qemu/target-sparc/exec.h 2007-03-19 09:22:48.763910000 -0400 > @@ -3,11 +3,7 @@ > #include "dyngen-exec.h" > #include "config.h" ^^^^^^^^ You missed a bit... Thiemo