Re: Adding support for top grub menu

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Jarno Suni wrote: > 2010/5/9 Vladimir 'φ-coder/phcoder' Serbinenko : > > >> I don't understand what you're exactly saying but if you install >> os-prober grub will discover other OS in update-grub. >> > > But if you use os-prober, I suppose it finds all kernels installed in > all distributi

Re: [PATCH] Fix grub.d/10_windows for Cygwin 1.7

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Christian Franke wrote: >> - >> + test -z "$needmap" || cat<> +drivemap -s (hd0) \$root >> +EOF >> You can't be sure on script runtime that C: is on hd0. Ironically >> especially when drivemap is used. >> But drivemap -s (hd0) (hd0) is a nop, so no harm done. >> On the other hand Vista and 7

Re: Adding support for top grub menu

2010-05-09 Thread Jarno Suni
2010/5/9 Vladimir 'φ-coder/phcoder' Serbinenko : > I don't understand what you're exactly saying but if you install > os-prober grub will discover other OS in update-grub. But if you use os-prober, I suppose it finds all kernels installed in all distributions and makes the grub menu too big. And

Re: [PATCH] Fix grub.d/10_windows for Cygwin 1.7

2010-05-09 Thread Christian Franke
Vladimir 'φ-coder/phcoder' Serbinenko wrote: Christian Franke wrote: This change is already included in Cygwin package grub-1.98-2. - + test -z "$needmap" || cat< This is already considered in the script: $needmap is set if ntldr is found but not if bootmgr is found. -- Reg

Re: Adding support for top grub menu

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Jarno Suni wrote: > I have many operating systems in my computer and I use a top grub menu > in a separate partition to chainload boot loaders of operating > systems. I think that practice is not supported well. I have made the > top menu by editing the grub.cfg of the dedicated partition manually

Re: [PATCH] Function parameters support

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
BVK Chaitanya wrote: > Hi, > > > Attached patch adds function parameters ($1, $2, etc. and $#) support > to GRUB script.. Shell expansion for "$@" and $@ specials is not yet > available. > > PS: It needs to be applied over my earlier, remove grub_script_cmdblock > patch. > > + grub_e

Re: [Patch] [bug #26237] multiple problems with usb devices

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Aleš Nesrsta wrote: >>> Additionally, specification says that bit 31 (FIT) should be set to one >>> if any change of FRAME_INTERVAL occurs - but I think it is necessary >>> only if OHCI is in operational state - which is not in our case. >> It can be done. Thanks. > As I wrote, it is most probably

Re: [PATCH] Fix grub.d/10_windows for Cygwin 1.7

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Christian Franke wrote: > This change is already included in Cygwin package grub-1.98-2. > - + test -z "$needmap" || cat < -- > Regards, > Christian Franke > > > > ___ > Grub-deve

Re: [PATCH] Remove grub_script_cmdblock

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
BVK Chaitanya wrote: > Hi, > > > Attached patch removes grub_script_cmdblock struct from GRUB script > code. It is useless and doesn't provide any benefits. Also, > functions that deal with list of commands are now renamed to use > _cmdlist suffix. > > ok for exp > > ---

Re: [PATCH] continue command support

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
BVK Chaitanya wrote: > Hi, > > > Attached patch provides "continue [n]" command support for GRUB > script. It needs to be applied over my earlier break command patch. > > Can go into exp too > > > >

Re: [PATCH] break command support

2010-05-09 Thread Vladimir 'φ-coder/phcoder' Serbinenko
Other than a comment about the case n> active_loops it can go into exp BVK Chaitanya wrote: > Hi, > > > Attached patch adds "break [n]" command support to GRUB script. This > needs to be applied on top of func-params patch. > > > > ---