Re: [PATCH] Tweak boot device detection for EFI and PPC.

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

 



On Thursday, March 19 2009, David Lehman said:
> According to my brief research, Apple bootstrap partitions should be
> of a size not less than 800KB and not more than 1MB.

That sounds right -- we just never let you make a partition < 1 MB in
the past.  Syntax looks okay, but as my other message said, I'd lean
towards keeping efi and apple boostrap

> @@ -254,7 +254,8 @@ class NewWorldPPC(PPC):
>  
>          for part in self.anaconda.id.storage.partitions:
>              # XXX do we need to also check the size?
> -            if part.format.type == "hfs" and part.bootable:
> +            if part.format.type == "hfs" and \
> +               (800.0 / 1024.00) <= part.size <= 1:
>                  bootDev = part

Does it make sense to have a validBootPartSize() method for the platform
object?  It'd save us from having this comparison in more than one place
and I know a lot of the arches end up having such a requirement.

Jeremy

_______________________________________________
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