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

Bug#502446: grub-installer patch with complete po file updates :-)



Hi,

Current grub-installer script will overwrite boot sector of wrong
partition for Intel based Mac users.  This is not so nice thing to have
for release.

I now have tested and updated patch for this bug and test d-i image at:
 http://people.debian.org/~osamu/d-i-osamu-grub2.iso
 http://people.debian.org/~osamu/grub-installer_1.35.all-but-po.diff.gz

It works!

I also made full patch including translated text for all languages and
put at: 
  http://people.debian.org/~osamu/grub-installer_1.35.all.diff.gz 
      (No code change but just po and comment out of set -x.)

So this patch will not degrade translation status, and please consider
this fr inclusion.  Patch is based on unstable 1.35 package.

---
Here is details.

Since my bug report and my patch concept, I spend some time to recall
d-i testing tricks.  (I made diunpk and dipk script:
http://wiki.debian.org/DebianInstaller/Modify/CD )

My initial patch had typo but my idea was confirmed right.
 1) GRUB2 device name has no offset and need adjustment as proposed.
 2) rescue mode now offer boot loader installation on GRUB 2 too.
 3) 30_os-prober script of grub-pc is fine but it must be run after
    rebooting system to be more useful. (see below)
 4) gptsync needs to be installed and run from shell console in chroot
    but this can be done by rEFIt chain bootloader side too.  Thus d-i
    is usable on MacTel (Intel Macs).

    I intentionally left not-so-precise statement of 
    "first extended partition" under GPT.  This is because 
    this will create translation headache.

Osamu 

FYI: If os-prober package is initially installed and a post reboot hook
     script is made to run update-grub script, grub.cfg is generated
     correctly without 30_otheros from grub-installer.  This will make
     cleaner installation code.  This should be post-lenny thing.

Attachment: grub-installer_1.35.all-but-po.diff.gz
Description: Binary data


Reply to: