Re: Fixing evolution-data-server on core-updates

[Kei Kebreau <kkebreau@posteo.net>]

Ludovic Courtès <ludo@gnu.org> writes:

> Hello Timothy,
>
> Timothy Sample <samplet@ngyro.com> skribis:
>
>> From bcd753f777687c52bba6b9bf4184879e69990118 Mon Sep 17 00:00:00 2001
>> From: Timothy Sample <samplet@ngyro.com>
>> Date: Sun, 14 Jul 2019 23:47:44 -0400
>> Subject: [PATCH] gnu: evolution-data-server: Fix locale issue.
>>
>> * gnu/packages/gnome.scm (evolution-data-server)[arguments]: Add a phase
>> that patches the source code to fix a locale issue.
>> ---
>>  gnu/packages/gnome.scm | 15 +++++++++++++++
>>  1 file changed, 15 insertions(+)
>>
>> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
>> index eb251498e9..a34adc4006 100644
>> --- a/gnu/packages/gnome.scm
>> +++ b/gnu/packages/gnome.scm
>> @@ -5168,6 +5168,21 @@ Exchange, Last.fm, IMAP/SMTP, Jabber, SIP and Kerberos.")
>>                             "tests/libedata-cal/test-cal-cache-utils.c")
>>                (("/bin/rm") (which "rm")))
>>              #t))
>> +         ;; This phase fixes locale canonicalization and prevents a few test
>> +         ;; failures.  The bug has been reported upstream:
>> +         ;; <https://gitlab.gnome.org/GNOME/evolution-data-server/issues/137>.
>> +         (add-after 'unpack 'patch-locale-canonicalization
>> +           (lambda _
>> +             (substitute* "src/libedataserver/e-collator.c"
>> +               (("len = uloc_canonicalize \\(posix_locale,.*" x)
>> +                ((lambda (xs) (string-join xs "\n" 'suffix))
>> +                 (list
>> +                  "if (g_ascii_strcasecmp(posix_locale, \"C\") == 0 ||"
>> +                  "    g_ascii_strcasecmp(posix_locale, \"POSIX\") == 0) {"
>> +                  "  posix_locale = \"en_US_POSIX\";"
>> +                  "}"
>> +                  x))))
>
> LGTM, thanks for fixing it!
>
> Ludo’.

Just FYI, I can confirm that this patch allows me to re-enable tests
that previously failed on core-updates!  I've re-enabled the relevant
failing tests on my own core-updates branch in anticipation of this
patch.  Thanks from me, too!

Kei