[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: gettext patch (beta)

From: phcoder
Subject: Re: gettext patch (beta)
Date: Sat, 11 Apr 2009 00:47:53 +0200
User-agent: Thunderbird (X11/20090318)

Hello, thanks for your work. It's a nice stuff, however it has some minor problems
Carles Pina i Estany wrote:
-Copy to /usr/share/locale/ca/LC_MESSAGES/
Languages files should go to a subdir of $PREFIX. E.g. to $PREFIX/langs/$
linux directories may be inaccessible
I need to write the (see po/TODO :-( ). I'm not used or
familiar to write Makefiles :-( if someone wants to help it would speed up the process quite much. It needs only to merge the files with
the new .pot, compile (msgfmt), and install to the correct directory.
Use common.rmk, don't write directly to Makefile.

I exactly know what has to do, so if someone knows about
installation/Makefiles and doesn't know about gettext it's not a
problem, contact me. Else I will try to implement soon.
I can help you, I'm not a makefile expert but can be useful

I would even invite to a couple of beers in Fosdem if someone does
this part :-)

-and more testing about 00_header with gettext detection.
-Add _("") for mainly all strings (I would do in a separate patch)
-I have seen that Grub2 is not printing correctly the accents,
could be a problem in gettext or in some other layer
Did you load unifont as your font? Are you in gfxterm mode? Plain pc console can't output unicode characters because it uses fixed-width 8-bit font. Perhaps loading the characters most useful for current languages to the upper 128 characters would be an option. OR we can just tell everyone to use gfxterm
Index: conf/
--- conf/      (revision 1952)
+++ conf/      (working copy)
Don't include auto-generated files in your patch
Index: gettext/gettext.c
--- gettext/gettext.c   (revision 0)
+++ gettext/gettext.c   (revision 0)
+static int
+grub_gettext_get_info (int offset)
+  int buf;
Use grub_uint32_t here. Also be aware of endianness. It should be
static grub_uint32_t
grub_gettext_get_info (int offset)
  grub_uint32_t buf;

  grub_file_seek (fd_mo, offset);
  grub_file_read (fd_mo, (char*) &buf, sizeof (buf));
  buf = grub_cpu_to_le32 (buf);
  return buf;
Same applies multiple times in different places.
grub_gettext_translation_number is a bit a misnomer because this name would suggest transforming translation into number
+  offsettranslation = grub_gettext_get_info (GETTEXT_OFFSET_TRANSLATION);
+  position=offsettranslation+i*8;
Please respect GCS. This should be position = offsettranslation + i * 8;
+  ret = grub_malloc(grub_strlen(orig) + 1);
+  grub_strcpy(ret,orig);
+  return ret;
This would fail if the string isn't present at all in .mo
+  if (magic != 0x950412de)
A define instead of hardcoded number is suggested
+  locale_prefix = grub_env_get ("locale_prefix");
You need to treat the case when no locale_prefix is defined. I suggest to put a default $prefix/locale
+  grub_sprintf (mo_file, "%s/%s/LC_MESSAGES/", locale_prefix, lang);
+  /* XXX: lang is written by the user, need to sanitaze the input?  */
I suggest
grub_sprintf (mo_file, "%s/", locale_prefix, lang);
because .mo need to reside together with grub so all LC_MESSAGE is just unnecessary


Vladimir 'phcoder' Serbinenko

reply via email to

[Prev in Thread] Current Thread [Next in Thread]