Unset autopart flag after running ksdata.autopart.execute since we
are still techically doing custom partitioning.

Don't show devices queued for removal in the custom spoke.

Don't include Roots whose root device has been queued for removal.
---
pyanaconda/ui/gui/spokes/custom.py | 36 +++++++++++++++++++++++++++++-------
1 files changed, 29 insertions(+), 7 deletions(-)

diff --git a/pyanaconda/ui/gui/spokes/custom.py b/pyanaconda/ui/gui/spokes/custom.py
index a97b96e..e304854 100644
--- a/pyanaconda/ui/gui/spokes/custom.py
+++ b/pyanaconda/ui/gui/spokes/custom.py
@@ -38,6 +38,7 @@ P_ = lambda x, y, z: gettext.ldngettext("anaconda", x, y, z)
from pyanaconda.product import productName, productVersion
from pyanaconda.storage.formats import device_formats
from pyanaconda.storage.size import Size
+from pyanaconda.storage import Root

from pyanaconda.ui.gui import UIObject
from pyanaconda.ui.gui.spokes import NormalSpoke
@@ -243,11 +244,18 @@ class CustomPartitioningSpoke(NormalSpoke):
# We can only have one page expanded at a time.
did_expand = False

+ unused = self._unusedDevices()
+ new_devices = [d for d in self.storage.devicetree.leaves if d not in unused and not d.exists]
+
+ new_install_name = _("New %s %s Installation") % (productName, productVersion)
+
+ roots = self.storage.roots[:]
+
# If we've not yet run autopart, add an instance of CreateNewPage. This
# ensures it's only added once.
- if not self._ran_autopart:
+ if not self._ran_autopart and not new_devices:
page = CreateNewPage(self.on_create_clicked)
- page.pageTitle = _("New %s %s Installation") % (productName, productVersion)
+ page.pageTitle = new_install_name
self._accordion.addPage(page, cb=self.on_page_clicked)
self._accordion.expandPage(page.pageTitle)
did_expand = True
@@ -255,18 +263,32 @@ class CustomPartitioningSpoke(NormalSpoke):
self._partitionsNotebook.set_current_page(0)
label = self.builder.get_object("whenCreateLabel")
label.set_text(self._when_create_text % (productName, productVersion))
+ elif new_devices:
+ swaps = [d for d in new_devices if d in self.storage.swaps]
+ mounts = dict([(d.format.mountpoint, d) for d in new_devices if getattr(d.format, "mountpoint", None)])
+ new_root = Root(mounts=mounts, swaps=swaps, name=new_install_name)
+ roots.insert(0, new_root)

# Add in all the existing (or autopart-created) operating systems.
- for root in self.storage.roots:
+ for root in roots:
+ if root.device and root.device not in self.storage.devices:
+ continue
+
page = Page()
page.pageTitle = root.name

- for swap in root.swaps:
- selector = page.addDevice("Swap", swap.size, None, self.on_selector_clicked)
- selector._device = swap
+ for device in root.swaps:
+ if device not in self.storage.devices:
+ continue
+
+ selector = page.addDevice("Swap", device.size, None, self.on_selector_clicked)
+ selector._device = device
selector._root = root

for (mountpoint, device) in root.mounts.iteritems():
+ if device not in self.storage.devices:
+ continue
+
selector = page.addDevice(self._mountpointName(mountpoint) or device.format.name, device.size, mountpoint, self.on_selector_clicked)
selector._device = device
selector._root = root
@@ -279,7 +301,6 @@ class CustomPartitioningSpoke(NormalSpoke):
self._accordion.expandPage(root.name)

# Anything that doesn't go with an OS we understand? Put it in the Other box.
- unused = self._unusedDevices()
if unused:
page = UnknownPage()
page.pageTitle = _("Unknown")
@@ -542,6 +563,7 @@ class CustomPartitioningSpoke(NormalSpoke):
# FIXME: Handle all the autopart exns here.
self.data.autopart.autopart = True
self.data.autopart.execute(self.storage, self.data, self.instclass)
+ self.data.autopart.autopart = False

# Create a new Root object for the new installation and put it into
# the storage object. This means any boot-related devices we make for
--
1.7.7.6

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

> @@ -542,6 +563,7 @@ class CustomPartitioningSpoke(NormalSpoke):
> # FIXME: Handle all the autopart exns here.
> self.data.autopart.autopart = True
> self.data.autopart.execute(self.storage, self.data, self.instclass)
> + self.data.autopart.autopart = False
>
> # Create a new Root object for the new installation and put it into
> # the storage object. This means any boot-related devices we make for

I'd like to see a comment on this block so we know what's going on with
the autopart setting in the future.

- Chris

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list