On Sat, Apr 01, 2000 at 08:15:49PM +0900, OKUJI Yoshinori wrote:
> > This seems to be a change in behaviour, and gratuitous one, too, because it
> > worked before. Therefore I'm filing this as a bug to the Debian BTS. One
> > could argue that it's of higher severity, because it effectively made my
> > system unbootable, but there is a workaround (loading a kernel image from
> > <1024 cyls).
> 
>   A person said that it was a bug for GRUB not to check the bitmap,
> and now you say that it is a bug for GRUB to check the bitmap. *sigh*

I'm not saying it's necessarily wrong to do that; I'm saying that if you do
that, it renders my system unbootable (and probably several others), which
is most certainly not the desired effect :)

> Clearly, the one which has a bug really is your BIOS. Not GRUB.

And I (blindly) believe you. But my system indeed can do LBA, and I wish
GRUB would see it too, somehow, without needing to recompile.

> Sorry, this is just a complaint.

Uh... of course it is a complaint. :)

> > Please provide a run-time switch/option to enable us people with Gigabyte
> > BX2000 motherboards with Award 4.51PG BIOSes, and >8GB hard drives to boot
> > properly :)
> 
>   Yeah. If you propose the interface, I'll implement it soon.

Any method would suit me, as long as I don't have to recompile. Not that
it's hard to recompile for me personally; but some other users might get
stuck in a situation like this.

Say, a `forcelba' option, or something like that.

> >  * I moved "@c %**end of header" magic line a little down in the file
> >    because I saw other documents use that (texinfo), and otherwise it seemed
> >    logical to include those tags in header, not text.
> 
>   But there is no benefit. The magic is effective for only a few
> macros.

True.

> The other changes are good.

Thanks.

BTW there's another small language issue in GRUB itself, with the setup
command. When it runs `install [...]' command, it displays "Run install
[...]", which is actually imperative, so one might think he then needs to
run it manually (presuming it doesn't display an error of some sort).

The following patch changes the printf to clearly indicate that the install
command is being executed (automatically).

-- 
Digital Electronic Being Intended for Assassination and Nullification
Index: builtins.c
===================================================================
RCS file: /home/cvs/grub/stage2/builtins.c,v
retrieving revision 1.51
diff -u -r1.51 builtins.c
--- builtins.c  2000/03/18 22:44:41     1.51
+++ builtins.c  2000/04/01 10:06:55
@@ -2705,7 +2705,7 @@
 #endif /* ! NO_BUGGY_BIOS_IN_THE_WORLD */
   
   /* Notify what will be run.  */
-  grub_printf (" Run \"install %s\"\n", cmd_arg);
+  grub_printf (" Running \"install %s\"...\n", cmd_arg);
 
   /* Make sure that SAVED_DRIVE and SAVED_PARTITION are identical
      with IMAGE_DRIVE and IMAGE_PARTITION, respectively.  */

Reply via email to