Re: [PATCH 02/30] Make the allocation and grow code optional.

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

 



> @@ -597,8 +603,10 @@ def doPartitioning(storage, exclusiveDisks=None):
>          bootDev.req_bootable = True
>  
>      # FIXME: make sure non-existent partitions have empty parents list
> -    allocatePartitions(disks, partitions)
> -    growPartitions(disks, partitions)
> +    if alloc:
> +        allocatePartitions(disks, partitions)
> +    if grow:
> +        growPartitions(disks, partitions)
>      # The number and thus the name of partitions may have changed now,
>      # allocatePartitions() takes care of this for new partitions, but not
>      # for pre-existing ones, so we update the name of all partitions here

This should probably be:

if alloc:
   allocatePartitions(disks, partitions)

   if grow:
      growPartitions(disks, partitions)

Otherwise it'd be possible to call with alloc=False, grow=True and that
just seems like asking for trouble.

- Chris

_______________________________________________
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