From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1DFDjW-0006OD-QO for mharc-grub-devel@gnu.org; Sat, 26 Mar 2005 11:01:54 -0500 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1DFDjU-0006Nt-Qf for grub-devel@gnu.org; Sat, 26 Mar 2005 11:01:52 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1DFDg7-0005w3-Rx for grub-devel@gnu.org; Sat, 26 Mar 2005 10:58:25 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DFDg0-0005fP-38 for grub-devel@gnu.org; Sat, 26 Mar 2005 10:58:16 -0500 Received: from [145.74.66.11] (helo=mail-cn.han.nl) by monty-python.gnu.org with esmtp (Exim 4.34) id 1DFD5m-00057A-GH for grub-devel@gnu.org; Sat, 26 Mar 2005 10:20:50 -0500 Received: from vscan-cn.han.nl (venus.han.nl [145.74.65.6]) by mail-cn.han.nl (Postfix) with ESMTP id 0BE2A9591 for ; Sat, 26 Mar 2005 16:20:50 +0100 (CET) Received: from mail-cn.han.nl ([145.74.66.11]) by vscan-cn.han.nl (venus.han.nl [145.74.65.6]) (amavisd-new, port 10024) with ESMTP id 06720-02 for ; Sat, 26 Mar 2005 16:20:48 +0100 (CET) Received: from mail1.han.nl (mail1.han.nl [145.74.103.11]) by mail-cn.han.nl (Postfix) with ESMTP id F3FEB93F7 for ; Sat, 26 Mar 2005 16:20:47 +0100 (CET) Received: from localhost.localdomain (mgerards.xs4all.nl [82.92.27.129]) by mail1.han.nl (Postfix) with ESMTP id BE585C045 for ; Sat, 26 Mar 2005 16:20:47 +0100 (CET) Mail-Copies-To: metgerards@student.han.nl To: The development of GRUB 2 References: <74cfc9a471d7669b38b24a5a89f35538@free.fr> <200503232145.04398.okuji@enbug.org> <1111616923.4241ed9b706fe@imp5-q.free.fr> <200503260122.10488.okuji@enbug.org> <1111838991.4245510f6e902@imp6-q.free.fr> <87eke2qztz.fsf@student.han.nl> <1111848950.424577f63f35d@imp6-q.free.fr> From: Marco Gerards Date: Sat, 26 Mar 2005 16:20:50 +0100 In-Reply-To: <1111848950.424577f63f35d@imp6-q.free.fr> (Antoine Terrienne's message of "Sat, 26 Mar 2005 15:55:50 +0100") Message-ID: <878y4aqvql.fsf@student.han.nl> User-Agent: Gnus/5.1007 (Gnus v5.10.7) Emacs/21.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Virus-Scanned: by amavisd-new (2.2.0) at vscan-cn.han.nl Subject: Re: Grub2 on Mac OS X X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 26 Mar 2005 16:01:53 -0000 Antoine Terrienne writes: Hi Antoine, >> If that is used, you could replace int32_t by grub_uint32_t, etc. > ok I was wondering if here were a specific reason for using both grub_int32_t > and int32_t types like one for the host part and one for the build part. But if > we can always use grub_xxx types it will make it a lot easier. The reason was that this code was part of another project. It has been put into GRUB 2 and I think I missed these int32_t thingies. >> I thought GRUBOF was not used anymore, but I see you replaced it. I >> hope I can find some time to have a look at your patch soon, but >> please understand if it will take a while. >> > > Yes GRUBOF have been removed from everywere exept in one .h file in the powerpc > part. Must have been forgotten. In fact many little problems like this one > seems to come from some great improvements made to Grub2-i386 that have not > been ported to the powerpc part. What kind of things do you think there are missing for the PPC port? It should be quite complete by now. It does not compile at the moment, but this will be fixed in CVS really soon. >> >> Cool! please check if your modification did not add any warnings or >> so, we would like to keep GRUB quite clean. >> > > most warnings come from the memcpy function that conflicts with the gcc builtin. Can you show us? >> A complete review has to follow... sorry... >> > > No problem there's no rush. > By the way I will probably put a more complete patch in a few days with all your > remarks. Cool. Thanks, Marco