[PATCH 2/4] Default to partitions for /boot* instead of just /boot/efi. (#884606)

David Lehman dlehman at redhat.com
Fri Dec 7 22:45:10 UTC 2012


There is some argument that grub2 does not do well with non-partition
/boot (https://bugzilla.redhat.com/show_bug.cgi?id=737508#c75).

Also, this change brings the defaults in line with our own automatic
partitioning.
---
 pyanaconda/ui/gui/spokes/custom.py |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/pyanaconda/ui/gui/spokes/custom.py b/pyanaconda/ui/gui/spokes/custom.py
index c87d039..542dc19 100644
--- a/pyanaconda/ui/gui/spokes/custom.py
+++ b/pyanaconda/ui/gui/spokes/custom.py
@@ -1766,11 +1766,9 @@ class CustomPartitioningSpoke(NormalSpoke, StorageChecker):
                                      AUTOPART_TYPE_PLAIN: DEVICE_TYPE_PARTITION,
                                      AUTOPART_TYPE_BTRFS: DEVICE_TYPE_BTRFS}
         device_type = device_type_from_autopart[self.data.autopart.type]
-        if device_type != DEVICE_TYPE_PARTITION and \
-             mountpoint == "/boot/efi":
-            device_type = DEVICE_TYPE_PARTITION
-        elif device_type != DEVICE_TYPE_PARTITION and \
-             fstype in partition_only_format_types:
+        if (device_type != DEVICE_TYPE_PARTITION and
+            (mountpoint.startswith("/boot") or
+             fstype in partition_only_format_types)):
             device_type = DEVICE_TYPE_PARTITION
 
         # some devices should never be encrypted
-- 
1.7.7.6



More information about the anaconda-patches mailing list