FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > ArchLinux > ArchLinux User Repository

 
 
LinkBack Thread Tools
 
Old 05-01-2010, 11:29 PM
David Cantrell
 
Default Clean up wording for oversized LVs (#587459)

---
iw/lvm_dialog_gui.py | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/iw/lvm_dialog_gui.py b/iw/lvm_dialog_gui.py
index ec043de..cffb68c 100644
--- a/iw/lvm_dialog_gui.py
+++ b/iw/lvm_dialog_gui.py
@@ -707,13 +707,14 @@ class VolumeGroupEditor:
templv.size = size
except ValueError:
self.intf.messageWindow(_("Not enough space"),
- _("The logical volumes you have "
- "configured require %(size)d MB,"
- " but the volume group only has "
- "%(tempvgsize)d MB. Please "
- "either make the volume group "
- "larger or make the logical "
- "volume(s) smaller.")
+ _("The size entered for this "
+ "logical volume (%(size)d MB) "
+ "exceeds the available space "
+ "of the volume group "
+ "(%(tempvgsize)d MB). Please "
+ "make the volume group larger "
+ "or make the logical volume "
+ "smaller.")
% {'size': size,
'tempvgsize': tempvg.size},
custom_icon="error")
--
1.6.6.1

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 05-03-2010, 08:49 AM
Hans de Goede
 
Default Clean up wording for oversized LVs (#587459)

Hi,

The new message suggests / could be read as the VG still having
tempvg.size free, but AFAIK tempvg.size is the VG size,
not the free space in the VG.

Regards,

Hans



On 05/02/2010 01:29 AM, David Cantrell wrote:

---
iw/lvm_dialog_gui.py | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/iw/lvm_dialog_gui.py b/iw/lvm_dialog_gui.py
index ec043de..cffb68c 100644
--- a/iw/lvm_dialog_gui.py
+++ b/iw/lvm_dialog_gui.py
@@ -707,13 +707,14 @@ class VolumeGroupEditor:
templv.size = size
except ValueError:
self.intf.messageWindow(_("Not enough space"),
- _("The logical volumes you have "
- "configured require %(size)d MB,"
- " but the volume group only has "
- "%(tempvgsize)d MB. Please "
- "either make the volume group "
- "larger or make the logical "
- "volume(s) smaller.")
+ _("The size entered for this "
+ "logical volume (%(size)d MB) "
+ "exceeds the available space "
+ "of the volume group "
+ "(%(tempvgsize)d MB). Please "
+ "make the volume group larger "
+ "or make the logical volume "
+ "smaller.")
% {'size': size,
'tempvgsize': tempvg.size},
custom_icon="error")


_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 05-04-2010, 04:46 AM
David Cantrell
 
Default Clean up wording for oversized LVs (#587459)

---
iw/lvm_dialog_gui.py | 16 +++++++++-------
1 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/iw/lvm_dialog_gui.py b/iw/lvm_dialog_gui.py
index ec043de..7d1a262 100644
--- a/iw/lvm_dialog_gui.py
+++ b/iw/lvm_dialog_gui.py
@@ -707,13 +707,15 @@ class VolumeGroupEditor:
templv.size = size
except ValueError:
self.intf.messageWindow(_("Not enough space"),
- _("The logical volumes you have "
- "configured require %(size)d MB,"
- " but the volume group only has "
- "%(tempvgsize)d MB. Please "
- "either make the volume group "
- "larger or make the logical "
- "volume(s) smaller.")
+ _("The size entered for this "
+ "logical volume (%(size)d MB) "
+ "combined with the size of the "
+ "other logical volume(s) "
+ "exceeds the size of the "
+ "volume group (%(tempvgsize)d "
+ "MB). Please make the volume "
+ "group larger or make the "
+ "logical volume smaller.")
% {'size': size,
'tempvgsize': tempvg.size},
custom_icon="error")
--
1.6.6.1

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 05-04-2010, 05:33 AM
David Cantrell
 
Default Clean up wording for oversized LVs (#587459)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Mon, 3 May 2010, Hans de Goede wrote:


The new message suggests / could be read as the VG still having
tempvg.size free, but AFAIK tempvg.size is the VG size,
not the free space in the VG.


Good point. I reworded it a bit and sent the updated patch as part of the
four patch series I posted today. However, I'm not really sure the rewording
clears up the confusion for users any less.


On 05/02/2010 01:29 AM, David Cantrell wrote:

---
iw/lvm_dialog_gui.py | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/iw/lvm_dialog_gui.py b/iw/lvm_dialog_gui.py
index ec043de..cffb68c 100644
--- a/iw/lvm_dialog_gui.py
+++ b/iw/lvm_dialog_gui.py
@@ -707,13 +707,14 @@ class VolumeGroupEditor:
templv.size = size
except ValueError:
self.intf.messageWindow(_("Not enough space"),
- _("The logical volumes you
have "
- "configured require %(size)d
MB,"
- " but the volume group only
has "
- "%(tempvgsize)d MB. Please
"
- "either make the volume
group "
- "larger or make the logical
"

- "volume(s) smaller.")
+ _("The size entered for this "
+ "logical volume (%(size)d
MB) "
+ "exceeds the available space
"

+ "of the volume group "
+ "(%(tempvgsize)d MB). Please
"
+ "make the volume group
larger "
+ "or make the logical volume
"

+ "smaller.")
% {'size': size,
'tempvgsize':
tempvg.size},

custom_icon="error")




- --
David Cantrell <dcantrell@redhat.com>

Red Hat / Honolulu, HI

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iEYEARECAAYFAkvfsaEACgkQ5hsjjIy1Vkno+ACg7TuW2s0nz1 +JuHN4KeNIdi+N
YFUAn2hUlqcifOfrDp8djQDRrB0BYhXl
=1BMn
-----END PGP SIGNATURE-----

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 05-04-2010, 07:53 AM
Hans de Goede
 
Default Clean up wording for oversized LVs (#587459)

Hi,

Better, but not really pretty. It would be better IMHO to just figure
out with some calculation how much free space is still available and
show that in the error dialog (don't we already calculate this somewhere?).

Anyways, I would prefer a better dialog, but if that turns out to be
hard to do, ACK.

Regards,

Hans


On 05/04/2010 06:46 AM, David Cantrell wrote:

---
iw/lvm_dialog_gui.py | 16 +++++++++-------
1 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/iw/lvm_dialog_gui.py b/iw/lvm_dialog_gui.py
index ec043de..7d1a262 100644
--- a/iw/lvm_dialog_gui.py
+++ b/iw/lvm_dialog_gui.py
@@ -707,13 +707,15 @@ class VolumeGroupEditor:
templv.size = size
except ValueError:
self.intf.messageWindow(_("Not enough space"),
- _("The logical volumes you have "
- "configured require %(size)d MB,"
- " but the volume group only has "
- "%(tempvgsize)d MB. Please "
- "either make the volume group "
- "larger or make the logical "
- "volume(s) smaller.")
+ _("The size entered for this "
+ "logical volume (%(size)d MB) "
+ "combined with the size of the "
+ "other logical volume(s) "
+ "exceeds the size of the "
+ "volume group (%(tempvgsize)d "
+ "MB). Please make the volume "
+ "group larger or make the "
+ "logical volume smaller.")
% {'size': size,
'tempvgsize': tempvg.size},
custom_icon="error")


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

Thread Tools




All times are GMT. The time now is 10:12 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org