FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.

» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora Packaging

LinkBack Thread Tools
Old 03-29-2010, 11:55 AM
Hans de Goede
Default Do not try to commit diskLabels on non partitionable devices (#576145)

There is no reason to call partedDisk.commit() when destroying
disklabels. partedDevice.clobber() is sufficient. More over calling
partedDisk.commit() causes issues, as we now recognize disk labels
on non partitionable devices and calling partedDisk.commit() on a non
partitionable device will always fails, as the device node will reject
the partition table reload ioctl.
storage/formats/disklabel.py | 4 ----
1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/storage/formats/disklabel.py b/storage/formats/disklabel.py
index 3e78f27..d4a706d 100644
--- a/storage/formats/disklabel.py
+++ b/storage/formats/disklabel.py
@@ -232,11 +232,7 @@ class DiskLabel(DeviceFormat):
if not os.access(self.device, os.W_OK):
raise DeviceFormatError("device path does not exist")

- self._partedDisk = parted.freshDisk(device=self.partedDevice,
- ty=self.partedDisk.type)
- self.commit()
- self.commit()
self.exists = False

def commit(self):

Anaconda-devel-list mailing list

Thread Tools

All times are GMT. The time now is 05:51 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org