Linux Archive

Linux Archive (http://www.linux-archive.org/)
-   Debian User (http://www.linux-archive.org/debian-user/)
-   -   Check for required space for / on live installs (#468867) (http://www.linux-archive.org/debian-user/186951-check-required-space-live-installs-468867-a.html)

Jeremy Katz 11-03-2008 03:12 PM

Check for required space for / on live installs (#468867)
 
We can actually check earlier on the live image how much space is needed
and so we can pass that along. This reuses a string from later in the file
so that we don't break the string freeze
---
partitions.py | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/partitions.py b/partitions.py
index a256e1a..baf8faf 100644
--- a/partitions.py
+++ b/partitions.py
@@ -1216,6 +1216,15 @@ class Partitions:
"megabytes which is usually too small to "
"install %s.") % (productName,))

+ if (slash and self.anaconda and
+ (slash.getActualSize(self, diskset) <
+ self.anaconda.backend.getMinimumSizeMB("/"))):
+ errors.append(_("Your %s partition is less than %s "
+ "megabytes which is lower than recommended "
+ "for a normal %s install.")
+ %("/", self.anaconda.backend.getMinimumSizeMB("/"),
+ productName))
+
def getBaseReqs(reqs):
n = 0
while not reduce(lambda x,y: x and (y.type not in [REQUEST_RAID, REQUEST_LV]),
--
1.6.0.3

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

Jeremy Katz 11-03-2008 05:24 PM

Check for required space for / on live installs (#468867)
 
On Mon, 2008-11-03 at 13:20 -0500, Chris Lumens wrote:
> This and the related patch look fine to me, except I like this wording
> for the error message:
>
> Your %s partition is less than the recommended %s megabytes needed
> for a normal %s install.

That'd break the string freeze, though. So not really an option right
now, although we can do it after we branch F10

Jeremy

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


All times are GMT. The time now is 07:39 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.