Re: String freeze break in gnome-disk-utility

2010-03-15 Thread David Zeuthen
On Mon, 2010-03-15 at 14:19 +0100, Claude Paroz wrote:
 Hi David,
 
 Your commit a60bbb6b in gnome-disk-utility [1] is breaking string
 freeze. Note that adding a new string and not marking it for translation
 is also considered a string freeze.
 
 So please ask for approval on this list or revert the change.
 Translators will be grateful  :-)
 
 [1]
 http://git.gnome.org/browse/gnome-disk-utility/commit/?id=a60bbb6b468e0fbc42680dd6b4d90997f314ffb4

OK, can I get an approval please? This is about the string %s File. An
appropriate translator comment would look like this.

 /* Translators: This is the name of a Drive backed by a file.
  * The %s is the size of the file (e.g. 42 GB or 5 KB).
  */
 g_string_append_printf (result, %s File, strsize);

Thanks,
David

___
gnome-i18n mailing list
gnome-i18n@gnome.org
http://mail.gnome.org/mailman/listinfo/gnome-i18n


Re: String freeze break in gnome-disk-utility

2010-03-15 Thread Claude Paroz
Le lundi 15 mars 2010 à 11:10 -0400, David Zeuthen a écrit :
 On Mon, 2010-03-15 at 14:19 +0100, Claude Paroz wrote:
  Hi David,
  
  Your commit a60bbb6b in gnome-disk-utility [1] is breaking string
  freeze. Note that adding a new string and not marking it for translation
  is also considered a string freeze.
  
  So please ask for approval on this list or revert the change.
  Translators will be grateful  :-)
  
  [1]
  http://git.gnome.org/browse/gnome-disk-utility/commit/?id=a60bbb6b468e0fbc42680dd6b4d90997f314ffb4
 
 OK, can I get an approval please? This is about the string %s File. An
 appropriate translator comment would look like this.
 
  /* Translators: This is the name of a Drive backed by a file.
   * The %s is the size of the file (e.g. 42 GB or 5 KB).
   */
  g_string_append_printf (result, %s File, strsize);

As we are closed to the release, I think you should tell us why this is
critical.
If this was not a bug, I suggest to postpone it for the .1 release. What
do you (or others) think?

Claude

___
gnome-i18n mailing list
gnome-i18n@gnome.org
http://mail.gnome.org/mailman/listinfo/gnome-i18n


Re: String freeze break in gnome-disk-utility

2010-03-15 Thread David Zeuthen
On Mon, 2010-03-15 at 16:25 +0100, Claude Paroz wrote:
  OK, can I get an approval please? This is about the string %s File. An
  appropriate translator comment would look like this.
  
   /* Translators: This is the name of a Drive backed by a file.
* The %s is the size of the file (e.g. 42 GB or 5 KB).
*/
   g_string_append_printf (result, %s File, strsize);
 
 As we are closed to the release, I think you should tell us why this is
 critical.
 If this was not a bug, I suggest to postpone it for the .1 release. What
 do you (or others) think?

I added this string because udisks starting exposing loop devices and
without this change the UI would say 10 GB Hard Disk for loop
devices... and this is a lot worse than saying 10 GB File even if it's
not translated. As such, I don't think it is wise to post-pone this
change. Normally, users will never deal with loop devices so it's not
like it's affecting a lot of users.

Thanks,
David


___
gnome-i18n mailing list
gnome-i18n@gnome.org
http://mail.gnome.org/mailman/listinfo/gnome-i18n


Re: String freeze break in gnome-disk-utility

2010-03-15 Thread Claude Paroz
Le lundi 15 mars 2010 à 11:57 -0400, David Zeuthen a écrit :
 On Mon, 2010-03-15 at 16:25 +0100, Claude Paroz wrote:
   OK, can I get an approval please? This is about the string %s File. An
   appropriate translator comment would look like this.
   
/* Translators: This is the name of a Drive backed by a file.
 * The %s is the size of the file (e.g. 42 GB or 5 KB).
 */
g_string_append_printf (result, %s File, strsize);
  
  As we are closed to the release, I think you should tell us why this is
  critical.
  If this was not a bug, I suggest to postpone it for the .1 release. What
  do you (or others) think?
 
 I added this string because udisks starting exposing loop devices and
 without this change the UI would say 10 GB Hard Disk for loop
 devices... and this is a lot worse than saying 10 GB File even if it's
 not translated. As such, I don't think it is wise to post-pone this
 change. Normally, users will never deal with loop devices so it's not
 like it's affecting a lot of users.

Thanks for the explanation. Approval 1 of 2 from i18n.

Claude

___
gnome-i18n mailing list
gnome-i18n@gnome.org
http://mail.gnome.org/mailman/listinfo/gnome-i18n


Re: String freeze break in gnome-disk-utility

2010-03-15 Thread Gil Forcada
El dl 15 de 03 de 2010 a les 17:21 +0100, en/na Claude Paroz va
escriure:
 Le lundi 15 mars 2010 à 11:57 -0400, David Zeuthen a écrit :
  On Mon, 2010-03-15 at 16:25 +0100, Claude Paroz wrote:
OK, can I get an approval please? This is about the string %s File. An
appropriate translator comment would look like this.

 /* Translators: This is the name of a Drive backed by a file.
  * The %s is the size of the file (e.g. 42 GB or 5 KB).
  */
 g_string_append_printf (result, %s File, strsize);
   
   As we are closed to the release, I think you should tell us why this is
   critical.
   If this was not a bug, I suggest to postpone it for the .1 release. What
   do you (or others) think?
  
  I added this string because udisks starting exposing loop devices and
  without this change the UI would say 10 GB Hard Disk for loop
  devices... and this is a lot worse than saying 10 GB File even if it's
  not translated. As such, I don't think it is wise to post-pone this
  change. Normally, users will never deal with loop devices so it's not
  like it's affecting a lot of users.
 
 Thanks for the explanation. Approval 1 of 2 from i18n.

2/2 then. Go ahead and commit ;)

Cheers,

 Claude
 
 ___
 gnome-i18n mailing list
 gnome-i18n@gnome.org
 http://mail.gnome.org/mailman/listinfo/gnome-i18n

-- 
gil forcada

[ca] guifi.net - una xarxa lliure que no para de créixer
[en] guifi.net - a non-stopping free network
bloc: http://gil.badall.net

___
gnome-i18n mailing list
gnome-i18n@gnome.org
http://mail.gnome.org/mailman/listinfo/gnome-i18n