Merge lp:~psusi/ubuntu/vivid/grub-installer/fix-efi-multi-disk-installs into lp:ubuntu/vivid/grub-installer
Proposed by
Phillip Susi
Status: | Merged |
---|---|
Merge reported by: | Phillip Susi |
Merged at revision: | not available |
Proposed branch: | lp:~psusi/ubuntu/vivid/grub-installer/fix-efi-multi-disk-installs |
Merge into: | lp:ubuntu/vivid/grub-installer |
Diff against target: |
31 lines (+13/-0) 2 files modified
debian/changelog (+7/-0) grub-installer (+6/-0) |
To merge this branch: | bzr merge lp:~psusi/ubuntu/vivid/grub-installer/fix-efi-multi-disk-installs |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mathieu Trudel-Lapierre | Needs Fixing | ||
Phillip Susi (community) | Needs Resubmitting | ||
Review via email: mp+247772@code.launchpad.net |
To post a comment you must log in.
There's a comment further down that explicitly mentions avoiding touching the active flag on GPT, so in that sense your changes look fine, but it seems the code that actually did ignore GPT from grub-installer itself went away, and now that is only checked in ensure-active itself -- could you move that comment and reword it so that the intent of the change is clear?
Something like "doing this because we won't set partitions active on GPT, and if $bootdisk points to a drive with no partition table things will explode in fun ways"... In your own words ;)