[master 3/3] Don't store size as int in SameSizeSet

vpodzime installerbot-noreply at redhat.com
Mon Oct 26 11:25:45 UTC 2015


From: Vratislav Podzimek <vpodzime at redhat.com>

This attribute is directly assigned to things like PartitionDevice.req_max_size
which should be an instance of Size. Also a general rule of thumb is to store
sizes as instances of Size so we should do it here too.

I don't really know why we ever converted the value to int() in that place, but
tests seem to pass just fine without the conversion so I guess everything's just
fine.
---
 blivet/partitioning.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/blivet/partitioning.py b/blivet/partitioning.py
index 6505825..6877cb7 100644
--- a/blivet/partitioning.py
+++ b/blivet/partitioning.py
@@ -1560,7 +1560,7 @@ def __init__(self, devices, size, grow=False, max_size=None):
 
             self.devices.append(partition)
 
-        self.size = int(size / len(devices))
+        self.size = size / len(devices)
         self.grow = grow
         self.max_size = max_size
 


-- 
To view this commit on github, visit https://github.com/rhinstaller/blivet/commit/4744841772f24edaef8ebc8788840fc9cecfe25d


More information about the anaconda-patches mailing list