On Mon, 2012-08-20 at 15:52 -0400, Chris Lumens wrote:
---
pyanaconda/image.py | 2 +-
pyanaconda/ui/gui/spokes/source.py | 45 ++++++++++++++++++++------------------
2 files changed, 25 insertions(+), 22 deletions(-)
diff --git a/pyanaconda/image.py b/pyanaconda/image.py
index 009e38a..0bc5c99 100644
--- a/pyanaconda/image.py
+++ b/pyanaconda/image.py
@@ -203,7 +203,7 @@ def opticalInstallMedia(devicetree, mountpoint=INSTALL_TREE):
# Return a list of Device instances that may have HDISO install media
# somewhere. Candidate devices are simply any that we can mount.
def potentialHdisoSources(devicetree):
- return filter(lambda d: d.format.mountable,
devicetree.getDevicesByType("partition"))
+ return filter(lambda d: d.exists and d.format.mountable,
devicetree.getDevicesByType("partition"))
This should probably check d.format.exists instead since the filesystem
needs to exist as well. There's no need to check the device's existence
separately AFAIK.
The rest looks okay to me.