Merge lp:~rcj/vmbuilder/automated-ec2-builds into lp:~ubuntu-on-ec2/vmbuilder/automated-ec2-builds

Proposed by Robert C Jennings
Status: Merged
Approved by: Mathieu Trudel-Lapierre
Approved revision: no longer in the source branch.
Merged at revision: 732
Proposed branch: lp:~rcj/vmbuilder/automated-ec2-builds
Merge into: lp:~ubuntu-on-ec2/vmbuilder/automated-ec2-builds
Diff against target: 25 lines (+0/-8)
1 file modified
ovf/part2disk-grubhelper.uefi (+0/-8)
To merge this branch: bzr merge lp:~rcj/vmbuilder/automated-ec2-builds
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre (community) Approve
Francis Ginther (community) Approve
Chris Glass Pending
Review via email: mp+329963@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote :

This looks correct based on the explanation and the IRC discussion.

review: Approve
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Looks good.

review: Approve
732. By Robert C Jennings

grub-efi: Remove unnecessary (and incorrect) file copy operations

grub EFI files were renamed, which has broken the copy operations.
These are unnecessary as the grub-install will perform all of the
needed file copies.

MP: https://code.launchpad.net/~rcj/vmbuilder/automated-ec2-builds/+merge/329963
[a=rcj][r=cyphermox,fginther]

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ovf/part2disk-grubhelper.uefi'
--- ovf/part2disk-grubhelper.uefi 2015-02-25 23:24:44 +0000
+++ ovf/part2disk-grubhelper.uefi 2017-08-30 20:20:44 +0000
@@ -223,9 +223,6 @@
223xchroot apt-get -y update223xchroot apt-get -y update
224224
225# Install the UEFI bits225# Install the UEFI bits
226xchroot cp /usr/lib/grub/x86_64-efi-signed/grubx64.efi.signed \
227 /boot/efi/EFI/BOOT/grubx64.efi
228
229BOOT_D="/boot/efi/EFI/BOOT"226BOOT_D="/boot/efi/EFI/BOOT"
230227
231# Make sure these modules are always pre-loaded228# Make sure these modules are always pre-loaded
@@ -238,11 +235,6 @@
238235
239EOF236EOF
240237
241
242xchroot cp /usr/lib/shim/shim.efi.signed ${BOOT_D}/shimx64.efi
243xchroot cp /usr/lib/shim/MokManager.efi.signed ${BOOT_D}/MokManager.efi
244xchroot cp /usr/lib/grub/x86_64-efi-signed/grubx64.efi.signed ${BOOT_D}/grubx64.efi
245
246xchroot grub-install /dev/${bn_lodev} \238xchroot grub-install /dev/${bn_lodev} \
247 --boot-directory=/boot \239 --boot-directory=/boot \
248 --efi-directory=/boot/efi \240 --efi-directory=/boot/efi \

Subscribers

People subscribed via source and target branches