Fixing evolution-data-server on core-updates

Fixing evolution-data-server on core-updates

[Timothy Sample]

[-- Attachment #1: Type: text/plain, Size: 541 bytes --]

Hi all,

While testing core-updates I found that evolution-data-server does not
build due to test failures.  The tests fail because
evolution-data-server does not accommodate newer versions of ICU.
Here’s the upstream bug report [1].  I’ve attached a patch that uses
“substitute*” to work around the problem (it’s rather simple).  I think
we should wait to hear from upstream, and if they don’t get to it in a
few days, use the patch.

[1] https://gitlab.gnome.org/GNOME/evolution-data-server/issues/137


-- Tim


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: the patch --]
[-- Type: text/x-patch, Size: 1710 bytes --]

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))))
+             #t))
          (add-before 'configure 'dont-override-rpath
            (lambda _
              (substitute* "CMakeLists.txt"
-- 
2.22.0

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

[Ludovic Courtès]

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’.

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

[Kei Kebreau]

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

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

[Jonathan Brielmaier]

So am I right in the assumption to use the patch proposed at:
https://gitlab.gnome.org/GNOME/evolution-data-server/issues/137

Could you then prepare this patch for core-updates?

On 7/15/19 6:09 AM, Timothy Sample wrote:
> Hi all,
>
> While testing core-updates I found that evolution-data-server does not
> build due to test failures.  The tests fail because
> evolution-data-server does not accommodate newer versions of ICU.
> Here’s the upstream bug report [1].  I’ve attached a patch that uses
> “substitute*” to work around the problem (it’s rather simple).  I think
> we should wait to hear from upstream, and if they don’t get to it in a
> few days, use the patch.
>
> [1] https://gitlab.gnome.org/GNOME/evolution-data-server/issues/137
>
>
> -- Tim
>

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

[Timothy Sample]

Hello,

Kei Kebreau <kkebreau@posteo.net> writes:

> 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.
>>> ---
>>> [...]
>>
>> 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

I heard back from upstream, and the fix will be included from version
3.33.5.  In the meantime, I spruced up the comment and pushed this as
d619686250d8bb15bf67031f8ac80f9cfb400a26.  When we update to GNOME 3.34,
we can remove it again (I’m hoping the comment will be a sufficient
reminder).

Thanks for looking it over!


-- Tim