Re: [PATCH] Make sure /boot ends up on the same disk as Apple Bootstrap (#497390).

[David Lehman <dlehman redhat com>]

On Tue, 2009-05-05 at 15:25 -0400, Chris Lumens wrote:
> ---
>  storage/partitioning.py |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)

Bravo. Ship it.

> 
> diff --git a/storage/partitioning.py b/storage/partitioning.py
> index 3046eec..0d282a6 100644
> --- a/storage/partitioning.py
> +++ b/storage/partitioning.py
> @@ -727,7 +727,10 @@ def allocatePartitions(disks, partitions):
>                                                             best.getSize()))
>                  free = best
>  
> -            if free and _part.req_bootable:
> +            # For platforms with a fake boot partition (like Apple Bootstrap or
> +            # PReP) and multiple disks, we need to ensure the /boot partition
> +            # ends up on the same disk as the fake one.
> +            if free and (_part.req_bootable or getattr(_part.format, "mountpoint", "").startswith("/boot")):
>                  # if this is a bootable partition we want to
>                  # use the first freespace region large enough
>                  # to satisfy the request