[blivet:master 14/17] Set format's mountpoint if it has the mountpoint attribute.

mulhern amulhern at redhat.com
Tue Jan 6 20:38:16 UTC 2015


Checking whether the device is mountable is somewhat expensive. Also, it
seems like the value of mountable could change. It would be depressing to
have not set the mountpoint here, but to later encounter the mountable() but
alas, no mountpoint, situation.

I can not find a location where the existance of a mountpoint is taken
to indicate that the format is mountable.

Signed-off-by: mulhern <amulhern at redhat.com>
---
 blivet/__init__.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/blivet/__init__.py b/blivet/__init__.py
index 5c834f7..1359f09 100644
--- a/blivet/__init__.py
+++ b/blivet/__init__.py
@@ -2423,7 +2423,7 @@ class FSSet(object):
         del ftype
         del dtype
 
-        if device.format.mountable:
+        if hasattr(device.format, "mountpoint"):
             device.format.mountpoint = mountpoint
 
         device.format.options = options
-- 
1.9.3



More information about the anaconda-patches mailing list