From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1KTN6b-0000YH-Il for mharc-grub-devel@gnu.org; Wed, 13 Aug 2008 16:38:05 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KTN6Y-0000YC-O1 for grub-devel@gnu.org; Wed, 13 Aug 2008 16:38:02 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KTN6W-0000Xy-9r for grub-devel@gnu.org; Wed, 13 Aug 2008 16:38:01 -0400 Received: from [199.232.76.173] (port=43375 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KTN6W-0000Xv-42 for grub-devel@gnu.org; Wed, 13 Aug 2008 16:38:00 -0400 Received: from moutng.kundenserver.de ([212.227.126.186]:55200) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1KTN6V-0001ip-EH for grub-devel@gnu.org; Wed, 13 Aug 2008 16:37:59 -0400 Received: from [85.180.44.194] (e180044194.adsl.alicedsl.de [85.180.44.194]) by mrelayeu.kundenserver.de (node=mrelayeu8) with ESMTP (Nemesis) id 0ML31I-1KTN6U0s7F-0006hJ; Wed, 13 Aug 2008 22:37:58 +0200 From: Felix Zielcke To: The development of GRUB 2 In-Reply-To: <1218659496.25595.7.camel@fz.local> References: <1218558487.19045.8.camel@fz.local> <1218558734.19045.10.camel@fz.local> <87fxp9di59.fsf@xs4all.nl> <20080812224041.GB3141@thorin> <1218606980.4008.9.camel@fz.local> <87abfhb60p.fsf@xs4all.nl> <1218659496.25595.7.camel@fz.local> Content-Type: text/plain Date: Wed, 13 Aug 2008 22:38:03 +0200 Message-Id: <1218659883.25595.8.camel@fz.local> Mime-Version: 1.0 X-Mailer: Evolution 2.22.3.1 Content-Transfer-Encoding: 7bit X-Provags-ID: V01U2FsdGVkX1/W0DSr49xXymOkUGKOn0FCStjqbgbmJeFmEka HSJfvM+z6M7i1BUN3XXgEU1h49844b8KuB5BABcRyqXJlC29Pz JLfi79vSkjzJwcYFgoXbFFNCni1eClz X-detected-kernel: by monty-python.gnu.org: Linux 2.6? (barebone, rare!) Subject: Re: [PATCH] kern/err.c + disk/raid.c error handling fixes 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: Wed, 13 Aug 2008 20:38:03 -0000 Am Mittwoch, den 13.08.2008, 22:31 +0200 schrieb Felix Zielcke: > Am Mittwoch, den 13.08.2008, 11:47 +0200 schrieb Marco Gerards: > > Felix Zielcke writes: > > > > > On Tue, Aug 12, 2008 at 11:42:58PM +0200, Marco Gerards wrote: > > >> > * kern/err.c [GRUB_UTIL]: Include . > > > > > >> Please don't do this. Why do you want this? > > > > > > It's needed for fprintf, and fprintf is only useful for GRUB_UTIL. > > > > I understood that. With this I meant "changing the kernel for GRUB_UTIL" :) > > > > I just did it first that way, because we need a simple and small patch > for lenny (i.e. debian-release) > > If I do it like you prefer duplicating it in util/misc.c then I need to > remove the static from `int grub_error_stack_assert;' and having it > prefixed with extern on grub/err.h > > Yes this change is no problem, grub-probe now compiles fine > But grub-mkimage not: > > grub_mkimage-util_misc.o: In function `grub_print_error': > /home/fz/grub/grub2.svn/util/misc.c:315: undefined reference to `grub_error_pop' > /home/fz/grub/grub2.svn/util/misc.c:312: undefined reference to `grub_errno' > /home/fz/grub/grub2.svn/util/misc.c:313: undefined reference to `grub_errmsg' > /home/fz/grub/grub2.svn/util/misc.c:313: undefined reference to `grub_printf' > /home/fz/grub/grub2.svn/util/misc.c:315: undefined reference to `grub_error_pop' > /home/fz/grub/grub2.svn/util/misc.c:320: undefined reference to `grub_printf' Sorry just copied the function from kern/err.c to util/misc.c without changing grub_printf () to fprintf () so just ignore these 2 lines please. > collect2: ld returned 1 exit status > make: *** [grub-mkimage] Error 1 > > So for me it seems kern/err.c has to be either doubled for the utils or, > just like I did first, with just changing 2 grub_printf to fprintf for > GRUB_UTIL > > > > _______________________________________________ > Grub-devel mailing list > Grub-devel@gnu.org > http://lists.gnu.org/mailman/listinfo/grub-devel