Re: [libvirt] [PATCH] Be more clever and verbose about localization-initialization.

I'd argue _for_ starting up libvirtd in case of errorous LC_* info. Since it is not a user-facing application but a system daemon, I think the impact of wrong language is small, but the benefit of having the daemon starting realiably is quite high. To explain my case as example: I sshed into a server from a weirdly configured desktop that had LC_MESSAGE set to de_DE and ran my script. Even though I set LANG=C there, the daemon would fail. Once I found out, I unset LC_*, but it still failed since tmux/bash "reinherited" the LC_*-stuff from ssh for some reason. It was a mess to debug and in the end I had to unset the env variables on the exec line that I passed on... :-( Anyways, in case you prefer only the verbose error, I attached a "verbose only" version, without the fallback part. Cheers, Andreas P.S. I hope OWA behaves better than HTC's email-app yesterday. ________________________________________ Von: Daniel P. Berrange [berrange(a)redhat.com] Gesendet: Montag, 7. Oktober 2013 18:24 An: Fuchs, Andreas Cc: libvir-list(a)redhat.com Betreff: Re: [libvirt] [PATCH] Be more clever and verbose about localization-initialization. On Mon, Oct 07, 2013 at 06:09:21PM +0200, andreas.fuchs(a)sit.fraunhofer.de wrote: You mean if it is set to a language code which is invalid ? I think that should just print a suitable error message and then exit, rather than just switching to C locale Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|