diff options
author | Jinfeng Wang <jinfeng.wang.cn@windriver.com> | 2024-11-04 15:35:23 +0800 |
---|---|---|
committer | Steve Sakoman <steve@sakoman.com> | 2024-11-26 05:37:10 -0800 |
commit | 75215fbce67593a6812ef9596f1d712e2188ce1c (patch) | |
tree | 8a87bfd400b3def423c40a62d685fa9b71940184 /meta | |
parent | 70d7a401f740b4df1a26f3ae2a8a7807fdc81fb7 (diff) | |
download | poky-75215fbce67593a6812ef9596f1d712e2188ce1c.tar.gz |
glib-2.0: fix glib-2.0 ptest failure when upgrading tzdata2024b
Backport 3 patches [1][2][3] for gdatetime test to fix the ptest failure.
[1] https://github.com/GNOME/glib/commit/c0619f08e6c608fd6464d2f0c6970ef0bbfb9ecf
[2] https://github.com/GNOME/glib/commit/30e9cfa5733003cd1079e0e9e8a4bff1a191171a
[3] https://github.com/GNOME/glib/commit/fe2699369f79981dcf913af4cfd98b342b84a9c1
(From OE-Core rev: aaeac4978111fa4051296cb800251432dc02226a)
Signed-off-by: Jinfeng Wang <jinfeng.wang.cn@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit 0c8f87d5d4ec9f286b1e85d114cb9a728c1ff64b)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
Diffstat (limited to 'meta')
4 files changed, 203 insertions, 0 deletions
diff --git a/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0001.patch b/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0001.patch new file mode 100644 index 0000000000..e78d96c3ce --- /dev/null +++ b/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0001.patch | |||
@@ -0,0 +1,72 @@ | |||
1 | From 729d1cbcd599e848bee07c72a276ccc02a06ddc0 Mon Sep 17 00:00:00 2001 | ||
2 | From: "Rebecca N. Palmer" <rebecca_palmer@zoho.com> | ||
3 | Date: Fri, 11 Oct 2024 09:38:52 +0100 | ||
4 | Subject: [PATCH 1/3] gdatetime test: Do not assume PST8PDT was always exactly | ||
5 | -8/-7 | ||
6 | |||
7 | In newer tzdata, it is an alias for America/Los_Angeles, which has a | ||
8 | slightly different meaning: DST did not exist there before 1883. As a | ||
9 | result, we can no longer hard-code the knowledge that interval 0 is | ||
10 | standard time and interval 1 is summer time, and instead we need to look | ||
11 | up the correct intervals from known timestamps. | ||
12 | |||
13 | Resolves: https://gitlab.gnome.org/GNOME/glib/-/issues/3502 | ||
14 | Bug-Debian: https://bugs.debian.org/1084190 | ||
15 | [smcv: expand commit message, fix whitespace] | ||
16 | Signed-off-by: Simon McVittie <smcv@debian.org> | ||
17 | |||
18 | Upstream-Status: Backport | ||
19 | [https://github.com/GNOME/glib/commit/c0619f08e6c608fd6464d2f0c6970ef0bbfb9ecf] | ||
20 | |||
21 | Signed-off-by: Jinfeng Wang <jinfeng.wang.cn@windriver.com> | ||
22 | --- | ||
23 | glib/tests/gdatetime.c | 22 ++++++++++++++++------ | ||
24 | 1 file changed, 16 insertions(+), 6 deletions(-) | ||
25 | |||
26 | diff --git a/glib/tests/gdatetime.c b/glib/tests/gdatetime.c | ||
27 | index d46f653ac..2eefc4106 100644 | ||
28 | --- a/glib/tests/gdatetime.c | ||
29 | +++ b/glib/tests/gdatetime.c | ||
30 | @@ -2930,6 +2930,7 @@ test_posix_parse (void) | ||
31 | { | ||
32 | GTimeZone *tz; | ||
33 | GDateTime *gdt1, *gdt2; | ||
34 | + gint i1, i2; | ||
35 | |||
36 | /* Check that an unknown zone name falls back to UTC. */ | ||
37 | G_GNUC_BEGIN_IGNORE_DEPRECATIONS | ||
38 | @@ -2953,16 +2954,25 @@ test_posix_parse (void) | ||
39 | |||
40 | /* This fails rules_from_identifier on Unix (though not on Windows) | ||
41 | * but passes anyway because PST8PDT is a zone name. | ||
42 | + * | ||
43 | + * Intervals i1 and i2 (rather than 0 and 1) are needed because in | ||
44 | + * recent tzdata, PST8PDT may be an alias for America/Los_Angeles, | ||
45 | + * and hence be aware that DST has not always existed. | ||
46 | + * https://bugs.debian.org/1084190 | ||
47 | */ | ||
48 | tz = g_time_zone_new_identifier ("PST8PDT"); | ||
49 | g_assert_nonnull (tz); | ||
50 | g_assert_cmpstr (g_time_zone_get_identifier (tz), ==, "PST8PDT"); | ||
51 | - g_assert_cmpstr (g_time_zone_get_abbreviation (tz, 0), ==, "PST"); | ||
52 | - g_assert_cmpint (g_time_zone_get_offset (tz, 0), ==, - 8 * 3600); | ||
53 | - g_assert (!g_time_zone_is_dst (tz, 0)); | ||
54 | - g_assert_cmpstr (g_time_zone_get_abbreviation (tz, 1), ==, "PDT"); | ||
55 | - g_assert_cmpint (g_time_zone_get_offset (tz, 1), ==,- 7 * 3600); | ||
56 | - g_assert (g_time_zone_is_dst (tz, 1)); | ||
57 | + /* a date in winter = non-DST */ | ||
58 | + i1 = g_time_zone_find_interval (tz, G_TIME_TYPE_STANDARD, 0); | ||
59 | + /* approximately 6 months in seconds, i.e. a date in summer = DST */ | ||
60 | + i2 = g_time_zone_find_interval (tz, G_TIME_TYPE_DAYLIGHT, 15000000); | ||
61 | + g_assert_cmpstr (g_time_zone_get_abbreviation (tz, i1), ==, "PST"); | ||
62 | + g_assert_cmpint (g_time_zone_get_offset (tz, i1), ==, - 8 * 3600); | ||
63 | + g_assert (!g_time_zone_is_dst (tz, i1)); | ||
64 | + g_assert_cmpstr (g_time_zone_get_abbreviation (tz, i2), ==, "PDT"); | ||
65 | + g_assert_cmpint (g_time_zone_get_offset (tz, i2), ==,- 7 * 3600); | ||
66 | + g_assert (g_time_zone_is_dst (tz, i2)); | ||
67 | g_time_zone_unref (tz); | ||
68 | |||
69 | tz = g_time_zone_new_identifier ("PST8PDT6:32:15"); | ||
70 | -- | ||
71 | 2.34.1 | ||
72 | |||
diff --git a/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0002.patch b/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0002.patch new file mode 100644 index 0000000000..d97bec12d6 --- /dev/null +++ b/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0002.patch | |||
@@ -0,0 +1,65 @@ | |||
1 | From 2702d7ff73649c05297105bab8977e2a9d28fa0b Mon Sep 17 00:00:00 2001 | ||
2 | From: Simon McVittie <smcv@debian.org> | ||
3 | Date: Fri, 18 Oct 2024 11:03:19 +0100 | ||
4 | Subject: [PATCH 2/3] gdatetime test: Try to make PST8PDT test more obviously | ||
5 | correct | ||
6 | |||
7 | Instead of using timestamp 0 as a magic number (in this case interpreted | ||
8 | as 1970-01-01T00:00:00-08:00), calculate a timestamp from a recent | ||
9 | year/month/day in winter, in this case 2024-01-01T00:00:00-08:00. | ||
10 | |||
11 | Similarly, instead of using a timestamp 15 million seconds later | ||
12 | (1970-06-23T15:40:00-07:00), calculate a timestamp from a recent | ||
13 | year/month/day in summer, in this case 2024-07-01T00:00:00-07:00. | ||
14 | |||
15 | Signed-off-by: Simon McVittie <smcv@debian.org> | ||
16 | |||
17 | Upstream-Status: Backport | ||
18 | [https://github.com/GNOME/glib/commit/30e9cfa5733003cd1079e0e9e8a4bff1a191171a] | ||
19 | |||
20 | Signed-off-by: Jinfeng Wang <jinfeng.wang.cn@windriver.com> | ||
21 | --- | ||
22 | glib/tests/gdatetime.c | 15 +++++++-------- | ||
23 | 1 file changed, 7 insertions(+), 8 deletions(-) | ||
24 | |||
25 | diff --git a/glib/tests/gdatetime.c b/glib/tests/gdatetime.c | ||
26 | index 2eefc4106..728b87343 100644 | ||
27 | --- a/glib/tests/gdatetime.c | ||
28 | +++ b/glib/tests/gdatetime.c | ||
29 | @@ -2954,19 +2954,16 @@ test_posix_parse (void) | ||
30 | |||
31 | /* This fails rules_from_identifier on Unix (though not on Windows) | ||
32 | * but passes anyway because PST8PDT is a zone name. | ||
33 | - * | ||
34 | - * Intervals i1 and i2 (rather than 0 and 1) are needed because in | ||
35 | - * recent tzdata, PST8PDT may be an alias for America/Los_Angeles, | ||
36 | - * and hence be aware that DST has not always existed. | ||
37 | - * https://bugs.debian.org/1084190 | ||
38 | */ | ||
39 | tz = g_time_zone_new_identifier ("PST8PDT"); | ||
40 | g_assert_nonnull (tz); | ||
41 | g_assert_cmpstr (g_time_zone_get_identifier (tz), ==, "PST8PDT"); | ||
42 | /* a date in winter = non-DST */ | ||
43 | - i1 = g_time_zone_find_interval (tz, G_TIME_TYPE_STANDARD, 0); | ||
44 | - /* approximately 6 months in seconds, i.e. a date in summer = DST */ | ||
45 | - i2 = g_time_zone_find_interval (tz, G_TIME_TYPE_DAYLIGHT, 15000000); | ||
46 | + gdt1 = g_date_time_new (tz, 2024, 1, 1, 0, 0, 0); | ||
47 | + i1 = g_time_zone_find_interval (tz, G_TIME_TYPE_STANDARD, g_date_time_to_unix (gdt1)); | ||
48 | + /* a date in summer = DST */ | ||
49 | + gdt2 = g_date_time_new (tz, 2024, 7, 1, 0, 0, 0); | ||
50 | + i2 = g_time_zone_find_interval (tz, G_TIME_TYPE_DAYLIGHT, g_date_time_to_unix (gdt2)); | ||
51 | g_assert_cmpstr (g_time_zone_get_abbreviation (tz, i1), ==, "PST"); | ||
52 | g_assert_cmpint (g_time_zone_get_offset (tz, i1), ==, - 8 * 3600); | ||
53 | g_assert (!g_time_zone_is_dst (tz, i1)); | ||
54 | @@ -2974,6 +2971,8 @@ test_posix_parse (void) | ||
55 | g_assert_cmpint (g_time_zone_get_offset (tz, i2), ==,- 7 * 3600); | ||
56 | g_assert (g_time_zone_is_dst (tz, i2)); | ||
57 | g_time_zone_unref (tz); | ||
58 | + g_date_time_unref (gdt1); | ||
59 | + g_date_time_unref (gdt2); | ||
60 | |||
61 | tz = g_time_zone_new_identifier ("PST8PDT6:32:15"); | ||
62 | #ifdef G_OS_WIN32 | ||
63 | -- | ||
64 | 2.34.1 | ||
65 | |||
diff --git a/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0003.patch b/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0003.patch new file mode 100644 index 0000000000..dfeec97810 --- /dev/null +++ b/meta/recipes-core/glib-2.0/files/gdatetime-test-fail-0003.patch | |||
@@ -0,0 +1,63 @@ | |||
1 | From df251923fb2c80cc7acfe99aaf8c98d7d7cbd29f Mon Sep 17 00:00:00 2001 | ||
2 | From: Simon McVittie <smcv@debian.org> | ||
3 | Date: Fri, 18 Oct 2024 11:23:42 +0100 | ||
4 | Subject: [PATCH 3/3] gdatetime test: Fall back if legacy System V PST8PDT is | ||
5 | not available | ||
6 | |||
7 | On recent versions of Debian, PST8PDT is part of the tzdata-legacy | ||
8 | package, which is not always installed and might disappear in future. | ||
9 | Successfully tested with and without tzdata-legacy on Debian unstable. | ||
10 | |||
11 | Signed-off-by: Simon McVittie <smcv@debian.org> | ||
12 | |||
13 | Upstream-Status: Backport | ||
14 | [https://github.com/GNOME/glib/commit/fe2699369f79981dcf913af4cfd98b342b84a9c1] | ||
15 | |||
16 | Signed-off-by: Jinfeng Wang <jinfeng.wang.cn@windriver.com> | ||
17 | --- | ||
18 | glib/tests/gdatetime.c | 19 +++++++++++++++++-- | ||
19 | 1 file changed, 17 insertions(+), 2 deletions(-) | ||
20 | |||
21 | diff --git a/glib/tests/gdatetime.c b/glib/tests/gdatetime.c | ||
22 | index 728b87343..9e1acd097 100644 | ||
23 | --- a/glib/tests/gdatetime.c | ||
24 | +++ b/glib/tests/gdatetime.c | ||
25 | @@ -2931,6 +2931,7 @@ test_posix_parse (void) | ||
26 | GTimeZone *tz; | ||
27 | GDateTime *gdt1, *gdt2; | ||
28 | gint i1, i2; | ||
29 | + const char *expect_id; | ||
30 | |||
31 | /* Check that an unknown zone name falls back to UTC. */ | ||
32 | G_GNUC_BEGIN_IGNORE_DEPRECATIONS | ||
33 | @@ -2953,11 +2954,25 @@ test_posix_parse (void) | ||
34 | g_time_zone_unref (tz); | ||
35 | |||
36 | /* This fails rules_from_identifier on Unix (though not on Windows) | ||
37 | - * but passes anyway because PST8PDT is a zone name. | ||
38 | + * but can pass anyway because PST8PDT is a legacy System V zone name. | ||
39 | */ | ||
40 | tz = g_time_zone_new_identifier ("PST8PDT"); | ||
41 | + expect_id = "PST8PDT"; | ||
42 | + | ||
43 | +#ifndef G_OS_WIN32 | ||
44 | + /* PST8PDT is in tzdata's "backward" set, packaged as tzdata-legacy and | ||
45 | + * not always present in some OSs; fall back to the equivalent geographical | ||
46 | + * name if the "backward" time zones are absent. */ | ||
47 | + if (tz == NULL) | ||
48 | + { | ||
49 | + g_test_message ("Legacy PST8PDT time zone not available, falling back"); | ||
50 | + tz = g_time_zone_new_identifier ("America/Los_Angeles"); | ||
51 | + expect_id = "America/Los_Angeles"; | ||
52 | + } | ||
53 | +#endif | ||
54 | + | ||
55 | g_assert_nonnull (tz); | ||
56 | - g_assert_cmpstr (g_time_zone_get_identifier (tz), ==, "PST8PDT"); | ||
57 | + g_assert_cmpstr (g_time_zone_get_identifier (tz), ==, expect_id); | ||
58 | /* a date in winter = non-DST */ | ||
59 | gdt1 = g_date_time_new (tz, 2024, 1, 1, 0, 0, 0); | ||
60 | i1 = g_time_zone_find_interval (tz, G_TIME_TYPE_STANDARD, g_date_time_to_unix (gdt1)); | ||
61 | -- | ||
62 | 2.34.1 | ||
63 | |||
diff --git a/meta/recipes-core/glib-2.0/glib.inc b/meta/recipes-core/glib-2.0/glib.inc index 9e742c4d3b..6ab52481b1 100644 --- a/meta/recipes-core/glib-2.0/glib.inc +++ b/meta/recipes-core/glib-2.0/glib.inc | |||
@@ -226,6 +226,9 @@ SRC_URI = "${GNOME_MIRROR}/glib/${SHRT_VER}/glib-${PV}.tar.xz \ | |||
226 | file://0001-meson-Run-atomics-test-on-clang-as-well.patch \ | 226 | file://0001-meson-Run-atomics-test-on-clang-as-well.patch \ |
227 | file://0001-gio-tests-resources.c-comment-out-a-build-host-only-.patch \ | 227 | file://0001-gio-tests-resources.c-comment-out-a-build-host-only-.patch \ |
228 | file://skip-timeout.patch \ | 228 | file://skip-timeout.patch \ |
229 | file://gdatetime-test-fail-0001.patch \ | ||
230 | file://gdatetime-test-fail-0002.patch \ | ||
231 | file://gdatetime-test-fail-0003.patch \ | ||
229 | " | 232 | " |
230 | SRC_URI:append:class-native = " file://relocate-modules.patch \ | 233 | SRC_URI:append:class-native = " file://relocate-modules.patch \ |
231 | file://0001-meson.build-do-not-enable-pidfd-features-on-native-g.patch \ | 234 | file://0001-meson.build-do-not-enable-pidfd-features-on-native-g.patch \ |