Re: [PATCH] Don't remove partitions twice.

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

 





On 03/24/2009 10:44 AM, Joel Granados Moreno wrote:
We remove the partitions at the end of clearpart function when we
traverse the clearpart list and "Destroy" each partition.

Looks good,

Regards,

Hans

---
  storage/partitioning.py |    3 ---
  1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/storage/partitioning.py b/storage/partitioning.py
index fdaf360..95d6ec2 100644
--- a/storage/partitioning.py
+++ b/storage/partitioning.py
@@ -318,9 +318,6 @@ def clearPartitions(storage):
                  storage.destroyDevice(leaf)
                  devices.remove(leaf)

-        # FIXME: this should be taken care of by DeviceTree.removeDevice
-        #        or Storage.destroyDevice
-        part.partedPartition.disk.removePartition(part.partedPartition)
          log.debug("partitions: %s" % [p.getDeviceNodeName() for p in part.partedPartition.disk.partitions])
          disk_name = os.path.basename(part.partedPartition.disk.device.path)
          if disk_name not in disks:

_______________________________________________
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