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

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