Re: [rhel6-branch] Make sure /boot is on the same disk as prepboot (#727136)

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

 



On Thu, Mar 01, 2012 at 01:10:32PM +0100, Martin Gracik wrote:
> If there is no separate /boot partition, treat the
> root partition as the /boot partition, because that's
> where /boot will end up.
> ---
>  storage/partitioning.py |   11 ++++++++++-
>  1 files changed, 10 insertions(+), 1 deletions(-)
> 
> diff --git a/storage/partitioning.py b/storage/partitioning.py
> index a6a5f81..20d79af 100644
> --- a/storage/partitioning.py
> +++ b/storage/partitioning.py
> @@ -991,6 +991,15 @@ def allocatePartitions(storage, disks, partitions, freespace):
>  
>      removeNewPartitions(disks, new_partitions)
>  
> +    # Search for the /boot partition.
> +    # If there is not a separate /boot partition,
> +    # the /boot will end up on the / partition.
> +    # We need this later for apple's bootstrap and prepboot,
> +    # because this partition needs to be on the same disk
> +    # as bootstrap/prepboot partition.
> +    _mountpoints = [getattr(part.format, "mountpoint", "") for part in new_partitions]
> +    _boot_part = "/boot" if "/boot" in _mountpoints else "/"

This kind of syntax is hard to read. Please expand it to traditional if:
else: blocks.

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)

Attachment: pgpyo7yc3Gez3.pgp
Description: PGP signature

_______________________________________________
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