Re: [GIT PULL/RESEND] kernel message catalog patches

From: Linus Torvalds
Date: Mon Oct 27 2008 - 12:13:15 EST




On Mon, 27 Oct 2008, Alan Cox wrote:
>
> > And as for the actual explanations: either they need to be totally outside
> > the kernel (in a project of their own), or they'd need to be "kernel-doc"
> > style things that are _in_ the source code. Not in Documentation/. Not
> > separate from the printk() that they are associated with.
>
> You really don't want 32 languages in mixed left/right rendering with
> multiple fonts in your kernel source. At least not with most editing and
> viewing tools....

I do agree. Another issue is that quite often the actual line to be
printed is likely fairly short, and often in an error path (so it's
indented and in an inconvenient place), but the explanation - even for a
_single_ language - may well be pretty involved and long.

Which is why I think for something like this, it really needs to be
entirely outside. Because putting it inside the kernel simply isn't going
to result in anything really useful. Either it can be close to the message
and be updated reasonably (never mind that if we really do support
multiple languages, that's not going to happen anyway, even if it's
close), or it's going to be somewhere else and not be updated when changes
happen - and then it migth as well be a separate project.

The 'separate project' is especially appropriate since any parsing won't
be done by the kernel anyway, but by some ksyslog thing.

Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/