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

Bug#136312: boot-floppies: Support list of languages to use like glibc LANGUAGE varialbe



On Fri, 2002-03-01 at 13:40, Petter Reinholdtsen wrote: 
>  - Avoid code duplication in main.c, and standardize how the messages
>    catalog file is loaded.  Allow the language code string to be a colon
>    separated list.
>  - Modify boxChooseLanguageVariant() accept a list of language codes
>  - Change release_notes.c to use the same code path with and without
>    the language chooser.  Make it accept a list of language codes.
>  - In util.c, allow a language list when choosing download host.
>  - in norwegian.src, demonstrate how this new list should be used for
>    Norwegian bokmål and nynorsk.

In principle this is fine, but the changes to main.c and release_notes.c
are sufficiently inscrutable at a casual reading to make me a bit
nervous.

Would you mind sending a patch that contains only the changes necessary
for multiple language code handling, without any reorganisation of the
rest of the code?  I think that would make it easier for people to
review.

+  if ( LOAD_TRMFILE("/etc/messages.en") != 0 ||
+       LOAD_TRMFILE("/etc/messages.trm") != 0 ||
+       LOAD_TRMFILE(TRMBACKUP) != 0 )

That doesn't look quite right to me.

p.




Reply to: