[gobolinux-devel] [gobolinux-commits] tools/ConfigTools/bin GenGrubMenu

[Lucas C. Villa Real]

On Dec 2, 2007 4:03 AM, Jonas Karlsson <jonka750 at student.liu.se> wrote:
> CVSROOT:        /sources/goboscripts
> Module name:    tools
> Changes by:     Jonas Karlsson <mohjive>        07/12/02 12:03:03
>
> Modified files:
>         ConfigTools/bin: GenGrubMenu
>
> Log message:
>         Grub always number the disk it boots from as '0'. If we install grub to mbr we probably live somewhere on that disk
>
> CVSWeb URLs:
> http://cvs.savannah.gnu.org/viewcvs/tools/ConfigTools/bin/GenGrubMenu?cvsroot=goboscripts&r1=1.14&r2=1.15
>
> Patches:
> Index: GenGrubMenu
> ===================================================================
> RCS file: /sources/goboscripts/tools/ConfigTools/bin/GenGrubMenu,v
> retrieving revision 1.14
> retrieving revision 1.15
> diff -u -b -r1.14 -r1.15
> --- GenGrubMenu 2 Dec 2007 11:56:11 -0000       1.14
> +++ GenGrubMenu 2 Dec 2007 12:03:02 -0000       1.15
> @@ -124,7 +124,7 @@
>                 if len(l) > 1 and l[1] == '*' :
>                         bootableWindowsPartitions.append(devToGrub(l[0]))
>
> -       grubFormatTarget = devToGrub('/dev/'+targetPartition)
> +       grubFormatTarget = re.sub('[1-9],','0,',devToGrub('/dev/'+targetPartition))
>         swapPartition = getSwapPartition()
>
>         o = ''

Why was this commit necessary? I'm looking at bug #243, and it
happened because of this change -- reverting to the old
'devToGrub('/dev/'+targetPartition)' fixes that. What problems did you
have with the previous code?

-- 
Lucas
powered by /dev/dsp