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 | |
| 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>
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 \ |
