asemke added inline comments. INLINE COMMENTS
> yurchor wrote in kxmlgui_unittest.cpp:1089 > AFAIK this is broken on Qt level. Cf. calendar widget in KOrganizer > (Ukrainian) > > F7273188: cal.png <https://phabricator.kde.org/F7273188> > > https://bugs.kde.org/show_bug.cgi?id=256952 > > There are new specifiers for the primary and alternative month names in glibc > but Qt does not implement them (must be something like "LLLL YYYY" instead of > "MMMM YYYY" in formatting). Rafal Luzynski (the author of the glibc patch) in > a private message (2018-01-25) promised me to send the patch for Qt but this > has not happened yet. This is maybe not Qt but the locale defintion files which are probably distro specific. I just checked on SLES and on openSuse. for ukrainian: LC_TIME=uk_UA.UTF-8 locale mon січень;лютий;березень;квітень;травень;червень;липень;серпень;вересень;жовтень;листопад;грудень for russian: LC_TIME=ru_RU.UTF-8 locale mon Январь;Февраль;Март;Апрель;Май;Июнь;Июль;Август;Сентябрь;Октябрь;Ноябрь;Декабрь This is, except of capital letters for ru_RU, correct. On Mac I get however січня;лютого;березня;квітня;травня;червня;липня;серпня;вересня;жовтня;листопада;грудня and января;февраля;марта;апреля;мая;июня;июля;августа;сентября;октября;ноября;декабря which is possessive case and is wrong if you reffer to the name of the month itself. @aacid on your distribution you'll most probably get the same output as on my Mac. Under the assumption that Qt evaluates the locale definition files, your test will probably fail on other distributions which do this correctly. REPOSITORY R263 KXmlGui REVISION DETAIL https://phabricator.kde.org/D23119 To: aacid Cc: yurchor, apol, kde-frameworks-devel, asemke, LeGast00n, GB_2, michaelh, ngraham, bruns
