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: