Re: [PATCH] When creating free space, handle cases other than clearpart --drives=

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

 



On Mon, 2009-03-16 at 16:11 -0400, Chris Lumens wrote:
> ---
>  storage/partitioning.py |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/storage/partitioning.py b/storage/partitioning.py
> index f4c40f2..0fdd6b9 100644
> --- a/storage/partitioning.py
> +++ b/storage/partitioning.py
> @@ -44,7 +44,10 @@ def _createFreeSpacePartitions(anaconda):
>      # get a list of disks that have at least one free space region of at
>      # least 100MB
>      disks = []
> -    for disk in [d for d in anaconda.id.storage.disks if d.name in anaconda.id.storage.clearPartDisks]:
> +    for disk in anaconda.id.storage.disks:
> +        if anaconda.id.storage.clearPartDisks and disk not in anaconda.id.storage.clearPartDisks:
> +            continue
> +
>          partedDisk = disk.partedDisk
>          part = disk.partedDisk.getFirstPartition()
>          while part:
> @@ -161,7 +164,7 @@ def doAutoPartition(anaconda):
>      disks = []
>      devs = []
>  
> -    if anaconda.id.storage.doAutoPart or anaconda.isKickstart:
> +    if anaconda.id.storage.doAutoPart and not anaconda.isKickstart:
>          # kickstart uses clearPartitions even without autopart
>          clearPartitions(anaconda.id.storage)
>  

an updates.img made with this change has fixed my virt kickstart
attempts.  +1 from here on the functional side, I don't speak to the
code changes themselves.

-- 
Jesse Keating
Fedora -- Freedom² is a feature!
identi.ca: http://identi.ca/jkeating

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
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