[PATCH] Check the format of /boot when using EFI (#689996)

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

 



On EFI /boot needs to be on an extX filesystem.
Add the error to the partitioning sanity check.

Resolves: rhbz#689996

NOTE: This has been tested with EFI on USB boot media and DVD, as well as in a KVM
---
 platform.py |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/platform.py b/platform.py
index e139033..bdb2357 100644
--- a/platform.py
+++ b/platform.py
@@ -244,8 +244,14 @@ class EFI(Platform):
             if req.format.type != "efi":
                 errors.append(_("/boot/efi is not EFI."))
 
-        # Don't try to check the disklabel on lv's etc, using lv for /boot
-        # is already checked in the generic Platform.checkBootRequest()
+            # EFI also needs /boot to be on an extX partition, either as its own
+            # partition or with / on extX. Get the format of whatever /boot is on
+            mntDict = self._mntDict()
+            boot_device = mntDict.get("/boot", mntDict.get("/"))
+            boot_errors = Platform.checkBootRequest(self, boot_device)
+            errors += boot_errors
+
+        # Don't try to check the disklabel on lv's etc.
         partitions = []
         if req.type == "partition":
             partitions = [ req ]
-- 
1.7.4.4

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list


[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux