----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120281/#review66971 -----------------------------------------------------------
C.UTF-8 is not a standard locale, even if quite a number of distributions support it, but not all: https://bugzilla.redhat.com/show_bug.cgi?id=902094 https://sourceware.org/bugzilla/show_bug.cgi?id=17318 - Luigi Toscano On Set. 19, 2014, 1:18 p.m., Maximiliano Curia wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120281/ > ----------------------------------------------------------- > > (Updated Set. 19, 2014, 1:18 p.m.) > > > Review request for KDE Runtime. > > > Bugs: 329983 > http://bugs.kde.org/show_bug.cgi?id=329983 > > > Repository: kde-runtime > > > Description > ------- > > The test was expecting a specific locale. Everything seems to indicate > that any UTF-8 enabled locale would work for this testcases, so use > C.UTF-8 and avoid to have to depend on all locales. > > > Diffs > ----- > > kioslave/trash/tests/testtrash.cpp f8d85de4ea4351bf91cba5ff374ae0ee5b567d5d > > Diff: https://git.reviewboard.kde.org/r/120281/diff/ > > > Testing > ------- > > Patch applied in Debian since 2014-01-14 > > > Thanks, > > Maximiliano Curia > >
