[PATCH] Display newly created partitions without a mountpoint, too (#886039).

Vratislav Podzimek vpodzime at redhat.com
Thu Sep 26 15:39:03 UTC 2013


On Thu, 2013-09-26 at 10:43 -0400, Chris Lumens wrote:
> Swap, for instance, does not have a mountpoint.  So the first partition was
> always getting displayed on the right hand side when swap was made.  This
> probably applies to other non-mountpoint formats too.
> ---
>  pyanaconda/ui/gui/spokes/custom.py | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/pyanaconda/ui/gui/spokes/custom.py b/pyanaconda/ui/gui/spokes/custom.py
> index 2d784d1..4361c8a 100644
> --- a/pyanaconda/ui/gui/spokes/custom.py
> +++ b/pyanaconda/ui/gui/spokes/custom.py
> @@ -2051,7 +2051,7 @@ class CustomPartitioningSpoke(NormalSpoke, StorageChecker):
>  
>          self._devices = self.__storage.devices
>          if not self._error:
> -            self._do_refresh(mountpointToShow=mountpoint)
> +            self._do_refresh(mountpointToShow=mountpoint or fstype)
>          else:
>              self._do_refresh()
>          self._updateSpaceDisplay()
> @@ -2127,7 +2127,7 @@ class CustomPartitioningSpoke(NormalSpoke, StorageChecker):
>          if not page:
>              page = self._current_page
>  
> -        log.debug("show first mountpoint: %s", page.pageTitle)
> +        log.debug("show mountpoint: %s", page.pageTitle)
>          if not page.members:
>              self._clear_current_selector()
>              return
> @@ -2137,7 +2137,7 @@ class CustomPartitioningSpoke(NormalSpoke, StorageChecker):
>              return
>  
>          for member in page.members:
> -            if member.get_property("mountpoint") == mountpoint:
> +            if member.get_property("mountpoint").lower() == mountpoint.lower():
>                  self.on_selector_clicked(member)
>                  break
ACK.

-- 
Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic



More information about the anaconda-patches mailing list