From: "Brian C. Lane" <bcl@xxxxxxxxxx> _is_valid_format needs to also check the mountpoints to make sure they are valid. Without this it will accept a previously formatted EFI fs and let the install continue without having it mounted on /boot/efi --- pyanaconda/bootloader.py | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/pyanaconda/bootloader.py b/pyanaconda/bootloader.py index e3044cd..06e893d 100644 --- a/pyanaconda/bootloader.py +++ b/pyanaconda/bootloader.py @@ -522,6 +522,12 @@ class BootLoader(object): % (desc, device.format.type)) ret = False + if mountpoints and hasattr(device.format, "mountpoint") \ + and device.format.mountpoint not in mountpoints: + self.errors.append(_("%s must be mounted on one of %s.") + % (desc, ", ".join(mountpoints))) + ret = False + log.debug("_is_valid_format(%s) returning %s" % (device.name,ret)) return ret -- 1.7.7.6 _______________________________________________ Anaconda-devel-list mailing list Anaconda-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/anaconda-devel-list