[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#488111: support for installing GRUB without blocklists on GPT



On Thursday 26 June 2008, Robert Millan wrote:
> > I get the feeling that this patch is incomplete. Are some new files
> > missing?
> >
> > Some other files show changes while I don't think they currently
> > exist, but that's OK. They look to have been copied from existing,
> > similar files dealing with the bootable flag for DOS partitions.
>
> Yeah.  Sorry, I forgot to mention that.
[...]

OK. But my question really was about any _other_ new files missing. But 
looking at the patch again, I guess my feeling may just have been wrong.

The key to this working is VALID_FLAGS returning bios_grub.

I don't see any obvious major problems with the patch.

A few naming suggestions that IMO make the scripts better readable:
- s/toggle_bios/toggle_biosgrub/ (affects script directory name too)
- s/biosable/biosgrub/
- s/unbiosable/nobiosgrub/
- s/valid_bios/valid_biosgrub/

Should the description "BIOS boot area:" not also mention grub in some 
way? Something like "Reserve BIOS boot area for GRUB:" (although that may 
already be a bit long for this dialog)?

From your initial mail it looks like:
- this will only work with grub-pc
- no additional changes are needed in grub-pc
- as grub-installer already automatically selects grub-pc for gpt, I guess
  no additional changes are needed in grub-installer

Is all that correct?

Cheers,
FJP



Reply to: