From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33524) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bUXK0-0003ua-Um for qemu-devel@nongnu.org; Tue, 02 Aug 2016 06:53:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bUXJw-0003e1-0d for qemu-devel@nongnu.org; Tue, 02 Aug 2016 06:53:15 -0400 Message-ID: <1470135189.30562.89.camel@redhat.com> From: Gerd Hoffmann Date: Tue, 02 Aug 2016 12:53:09 +0200 In-Reply-To: <1470129427.12584.22.camel@kernel.crashing.org> References: <1469608053.5978.132.camel@kernel.crashing.org> <20160728055102.GE2588@voom.fritz.box> <1470080328.30562.70.camel@redhat.com> <1470086678.5978.373.camel@kernel.crashing.org> <1470115519.30562.80.camel@redhat.com> <1470116113.12584.10.camel@kernel.crashing.org> <1470128146.30562.88.camel@redhat.com> <1470129427.12584.22.camel@kernel.crashing.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Mime-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH] ppc: Add MacOS VGA driver ROM List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Benjamin Herrenschmidt Cc: David Gibson , Mark Cave-Ayland , qemu-ppc@nongnu.org, qemu-devel@nongnu.org, Alexander Graf On Di, 2016-08-02 at 19:17 +1000, Benjamin Herrenschmidt wrote: > On Tue, 2016-08-02 at 10:55 +0200, Gerd Hoffmann wrote: > > > CodeWarrior for MacOS X does come with command line versions of the > > > tools, I plan to look into doing a Makefile once I've sorted out the > > > various arguments for these things at least. > >=20 > > Yes, you do that once, put it into roms/Makefile, and the next time you > > just do "make -C roms macosdrivers" instead of sorting out the various > > arguments *again*. > >=20 > > Bonus: Any possible contributers will have an easier start too. >=20 > I did notice that having Makefiles in the roms directory make > git submodule barf ... I had to remove them for it to be able > to clone the submodules. I mean roms/Makefile, not roms/$submodule/Makefile cheers, Gerd