New: [rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

This commit complements blivet's
c35aafe4b27318faee52b5a0ea44ca9a6fd4a1ec because RHEL7 doesn't support
working with HFS+ partitions.

The HW boots just fine with VFAT EFI system partition. The downside of
using VFAT is that there aren't pretty menus, but I prefer no menus but
working system over non-working system each time.

[rhel7-branch 1/1] Do not create HFS+ EFI partition on Intel Macs

[dashea]

From: =?UTF-8?q?Jan=20Kundr=C3=A1t?=

This commit complements blivet's
c35aafe4b27318faee52b5a0ea44ca9a6fd4a1ec because RHEL7 doesn't support
working with HFS+ partitions.

The HW boots just fine with VFAT EFI system partition. The downside of
using VFAT is that there aren't pretty menus, but I prefer no menus but
working system over non-working system each time.
---
 pyanaconda/kickstart.py | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/pyanaconda/kickstart.py b/pyanaconda/kickstart.py
index 0d722d1..222c389 100644
--- a/pyanaconda/kickstart.py
+++ b/pyanaconda/kickstart.py
@@ -1119,11 +1119,8 @@ def execute(self, storage, ksdata, instClass):
             if self.onPart:
                 ksdata.onPart[kwargs["name"]] = self.onPart
         elif self.mountpoint == "/boot/efi":
-            if blivet.arch.isMactel():
-                ty = "hfs+"
-            else:
-                ty = "EFI System Partition"
-                self.fsopts = "defaults,uid=0,gid=0,umask=0077,shortname=winnt"
+            ty = "EFI System Partition"
+            self.fsopts = "defaults,uid=0,gid=0,umask=0077,shortname=winnt"
         else:
             if self.fstype != "":
                 ty = self.fstype

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

Related bug: https://bugzilla.redhat.com/show_bug.cgi?id=1219196

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

Added label: rhel7-branch.

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

Looks good to me.

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

Added label: ACK.

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

Closed.

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

Reopened.

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

This was auto-closed by github when I rebased rhel7-branch.  I'm reopening it so it can get the consideration it deserves.

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

Closed.

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

pushed.

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

@bcl , I don't see this commit in the rhel7-branch. Could you please check it once more?

[rhinstaller/anaconda/pulls/101 rhel7-branch] Do not create HFS+ EFI partition on Intel Macs

[dashea]

> I don't see this commit in the rhel7-branch. Could you please check it once more?

It's there:
https://github.com/rhinstaller/anaconda/blob/rhel7-branch/pyanaconda/kicksta...