xorat / grubutils

Automatically exported from code.google.com/p/grubutils
0 stars 0 forks source link

can not compile wee, it seems exist a bug #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.  cd grubutils/wee
2.  make

What is the expected output? What do you see instead?

wee63start.S:248: Error: value of 67240 too large for field of 2 bytes at 132

What version of the product are you using? On what operating system?

OS ArchLinux, gcc 4.6.1 
http://grubutils.googlecode.com/svn/grubutils/wee rev:82

Please provide any additional information below.

I tried to patch the source for remove unused variable here is my patch.

$ svn di
Index: builtins.c
===================================================================
--- builtins.c  (revision 86)
+++ builtins.c  (working copy)
@@ -1207,9 +1207,9 @@
 #endif
     if (grub_memcmp (arg, "--hook", 6) == 0)
       {
-   char *p;
+   /*dlin char *p; */

-   p = arg + 6;
+   /*dlin p = arg + 6; */
    unset_int13_handler (0);
    if (drive_map_slot_empty (bios_drive_map[0]))
        return ! (errnum = ERR_NO_DRIVE_MAPPED);
@@ -3378,7 +3378,7 @@

       {
 #if 1
-   unsigned long ret;
+   /*dlin unsigned long ret; */
    struct realmode_regs regs;

    regs.ebx = 0;   // start the iteration
@@ -3394,7 +3394,7 @@
      regs.ds = regs.es = regs.fs = regs.gs = regs.eflags = -1;
      regs.cs = -1;     // CS=0xFFFFFFFF to run an INT instruction
      regs.eip = 0xFFFF15CD;    // run int15
-     ret = realmode_run ((unsigned long)&regs);
+     /*dlin ret = */ realmode_run ((unsigned long)&regs);
      if (regs.eax != 0x534D4150)
      {
        grub_printf ("\nFatal: Your BIOS has no support for System Memory Map(INT15/EAX=E820h).\nAs a result you cannot use the --mem option.\n");

Original issue reported on code.google.com by dlin...@gmail.com on 4 Sep 2011 at 4:13

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r94.

Original comment by chenall.cn on 8 Dec 2011 at 12:58

GoogleCodeExporter commented 9 years ago
Updated.thank you.

Could you please help me to manage this source?

Original comment by chenall.cn on 8 Dec 2011 at 12:58

GoogleCodeExporter commented 9 years ago
This issue was updated by revision r105.

this issue is no longer reproducible with Arch Linux.
test environment:
Arch Linux 3.2.8-1-ARCH
Binutils 2.22.0.20111227
gcc version 4.6.2 20120120 (prerelease) (GCC)

Original comment by roy...@gmail.com on 29 Feb 2012 at 2:41