On 03/15/2009 08:21 AM, David Lehman wrote:
On Fri, 2009-03-13 at 18:09 -1000, David Cantrell wrote:
For autopartition requests as well as free space regions, set
exists=False when calling newPartition().
I think this was an attempt to work around that bug in
PartitionDeviceFactory that caused it to return None in some cases. If
so, let's get rid of it now that the bug's fixed. The constructor
defaults the argument to None, which should be sufficient in all cases.
Holding off on this one.
Dave
---
storage/partitioning.py | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/storage/partitioning.py b/storage/partitioning.py
index 47800da..7e84489 100644
--- a/storage/partitioning.py
+++ b/storage/partitioning.py
@@ -68,7 +68,8 @@ def _createFreeSpacePartitions(anaconda):
part = anaconda.id.storage.newPartition(fmt_type=fmt_type,
size=1,
grow=True,
- disks=[disk])
+ disks=[disk],
+ exists=False)
if part:
anaconda.id.storage.createDevice(part)
devs.append(part)
@@ -95,7 +96,8 @@ def _schedulePartitions(anaconda, disks):
grow=grow,
maxsize=maxsize,
mountpoint=mountpoint,
- disks=disks)
+ disks=disks,
+ exists=False)
# schedule the device for creation
anaconda.id.storage.createDevice(dev)
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
--
David Cantrell <dcantrell@xxxxxxxxxx>
Red Hat / Honolulu, HI
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
