Time format not taken from LC_TIME

just see https://bugs.edge.launchpad.net/indicator-datetime/+bug/579134/comments/19

You can discuss this next week with ted, I think

Trying to keep the LC_TIME talk here, I'll copy your message:

"it's unlikely LC_TIME will work as ted's approach is to realy on LANG locale and picking up a string from translation. If you run it under LANG="en_GB.UTF-8", that should work (even if it doesn't respect the spec)"

Well, no. (A) indicator-datetime does not have an en_GB translation and (B) the point of this bug is that it shouldn't be using LC_MESSAGES (which is what gettext uses) for time formatting, but rather LC_TIME.

It's perfectly reasonable to set one locale for message strings and another for time. The Language Support program explicitly allows you to do this via the GUI, so it's not just some hacker thing. I do it because I want US strings, but I like 24h time.

(B) I totally agree :) Just trying to explain to you why it didn't work.

Fwiw, I use en_DK and expect 24h time format with that, but I got 12h format.

@mterry: Your test program seems to give the correct result, but -- what will you do with the 24/12hr information in the end? There is "%R" for a locale-dependent 24h format without seconds but no equivalent for 12h that I'm aware of. Simply using "%I:%M %p" for 12hr locales would not make everyone happy.

I think using translations might actually work if you use a little hack, something along the lines of:
char *message_locale = setlocale(LC_MESSAGES, NULL);
char *time_locale = setlocale(LC_TIME, NULL);
setlocale(LC_MESSAGES, time_locale);
char *time_format = _("%I:%M %p"); /* This will be translated using the PO file corresponding to LC_TIME!*/
setlocale(LC_MESSAGES, message_locale);

Good point. Or even easier, using dcgettext, which lets you specify a category. I'll whip up a patch.

Oh, of course, that makes it much less hackish -- I was so used to the _("...") makro that I forgot the function call behind it ;)

Marcel, dcgetext didn't actually do what I wanted after all. I ended up using a modified version of your function that also made sure LANGUAGE was unset (because that trumps LC_*). I also used my function for determining 12h/24h and made a few more fixups. See this bug's merge request.

Natty's current package is 0.1.90.is.0.0.6-0ubuntu2 which does not contain this fix yet.