Merge lp:~tj/grub-installer/lp1354730 into lp:~ubuntu-core-dev/grub-installer/ubuntu
Proposed by
TJ
on 2015-02-18
| Status: | Merged |
|---|---|
| Approved by: | Mathieu Trudel-Lapierre on 2015-02-19 |
| Approved revision: | 1269 |
| Merged at revision: | 1269 |
| Proposed branch: | lp:~tj/grub-installer/lp1354730 |
| Merge into: | lp:~ubuntu-core-dev/grub-installer/ubuntu |
| Diff against target: |
27 lines (+8/-1) 2 files modified
debian/changelog (+7/-0) grub-installer (+1/-1) |
| To merge this branch: | bzr merge lp:~tj/grub-installer/lp1354730 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Mathieu Trudel-Lapierre | 2015-02-18 | Approve on 2015-02-19 | |
|
Review via email:
|
|||
Description of the Change
Fix installer failure due to grub-probe returning more than one device in the case of BTRS multi-disk RAID volumes.
To post a comment you must log in.
On 19/02/15 16:24, Mathieu Trudel-Lapierre wrote:
> Review: Approve
>
> Looks good, but the version number really should be 1.78ubuntu25, not 1.78ubuntu24.1. I'll fix that as I merge the changes to not further delay things :)
>
OK. I was going by Colin Watson's comment in the original M.P. #230222 where he said:
"Oh, also, the version number will need to be 1.78ubuntu20.1 rather than 1.78ubuntu21."
But I think now, looking at it, that was because the ...ubuntu21 had already been used in one of the release branches.

Looks good, but the version number really should be 1.78ubuntu25, not 1.78ubuntu24.1. I'll fix that as I merge the changes to not further delay things :)