[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
81/115: gnu: glib: Skip ‘testfilemonitor’ test.
From: |
guix-commits |
Subject: |
81/115: gnu: glib: Skip ‘testfilemonitor’ test. |
Date: |
Fri, 15 Oct 2021 09:49:13 -0400 (EDT) |
apteryx pushed a commit to branch core-updates-frozen-batched-changes
in repository guix.
commit 4b109af57362f579b4a00795e34b2fdb7633819c
Author: Thiago Jung Bauermann <bauermann@kolabnow.com>
AuthorDate: Tue Oct 12 13:03:12 2021 -0300
gnu: glib: Skip ‘testfilemonitor’ test.
On powerpc64le-linux, it fails approximately 2 out of 3 times. On x86_64 it
also fails every now and then, at least on glib’s CI.
* gnu/packages/patches/glib-disable-testfilemonitor-test.patch: New file.
* gnu/local.mk (dist_patch_DATA): Add new patch.
* gnu/packages/glib.scm: Use it.
Signed-off-by: Maxim Cournoyer <maxim.cournoyer@gmail.com>
---
gnu/local.mk | 1 +
gnu/packages/glib.scm | 3 +-
.../glib-disable-testfilemonitor-test.patch | 111 +++++++++++++++++++++
3 files changed, 114 insertions(+), 1 deletion(-)
diff --git a/gnu/local.mk b/gnu/local.mk
index 2ad4c2a..51c0641 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1137,6 +1137,7 @@ dist_patch_DATA =
\
%D%/packages/patches/glib-appinfo-watch.patch \
%D%/packages/patches/glib-networking-gnutls-binding.patch \
%D%/packages/patches/glib-skip-failing-test.patch \
+ %D%/packages/patches/glib-disable-testfilemonitor-test.patch \
%D%/packages/patches/glibc-CVE-2018-11236.patch \
%D%/packages/patches/glibc-CVE-2018-11237.patch \
%D%/packages/patches/glibc-CVE-2019-7309.patch \
diff --git a/gnu/packages/glib.scm b/gnu/packages/glib.scm
index 2083a17..f4dc0d0 100644
--- a/gnu/packages/glib.scm
+++ b/gnu/packages/glib.scm
@@ -191,7 +191,8 @@ shared NFS home directories.")
(base32 "0f1iprj7v0b5wn9njj39dkl25g6filfs7i4ybk20jq821k1a7qg7"))
(patches
(search-patches "glib-appinfo-watch.patch"
- "glib-skip-failing-test.patch"))
+ "glib-skip-failing-test.patch"
+ "glib-disable-testfilemonitor-test.patch"))
(modules '((guix build utils)))
(snippet
'(begin
diff --git a/gnu/packages/patches/glib-disable-testfilemonitor-test.patch
b/gnu/packages/patches/glib-disable-testfilemonitor-test.patch
new file mode 100644
index 0000000..3feaa75
--- /dev/null
+++ b/gnu/packages/patches/glib-disable-testfilemonitor-test.patch
@@ -0,0 +1,111 @@
+On powerpc64le-linux, testfilemonitor fails approximately 2 out of 3 times.
+Patch obtained from:
+
+https://sources.debian.org/patches/glib2.0/2.66.8-1/debian/testfilemonitor-Skip-if-we-are-avoiding-flaky-tests.patch/
+
+From: Simon McVittie <smcv@debian.org>
+Date: Tue, 25 Feb 2020 10:45:07 +0000
+Subject: testfilemonitor: Skip if we are avoiding flaky tests
+
+See https://gitlab.gnome.org/GNOME/glib/issues/1634
+
+Signed-off-by: Simon McVittie <smcv@debian.org>
+Forwarded: no
+---
+ gio/tests/testfilemonitor.c | 34 +++++++++++++++++++++++++++++++++-
+ 1 file changed, 33 insertions(+), 1 deletion(-)
+
+diff --git a/gio/tests/testfilemonitor.c b/gio/tests/testfilemonitor.c
+index b74dc2b..79b5776 100644
+--- a/gio/tests/testfilemonitor.c
++++ b/gio/tests/testfilemonitor.c
+@@ -21,6 +21,12 @@ setup (Fixture *fixture,
+ gchar *path = NULL;
+ GError *local_error = NULL;
+
++ if (g_getenv ("DEB_ALLOW_FLAKY_TESTS") == NULL)
++ {
++ g_test_skip ("https://gitlab.gnome.org/GNOME/glib/issues/1634");
++ return;
++ }
++
+ path = g_dir_make_tmp ("gio-test-testfilemonitor_XXXXXX", &local_error);
+ g_assert_no_error (local_error);
+
+@@ -37,7 +43,9 @@ teardown (Fixture *fixture,
+ {
+ GError *local_error = NULL;
+
+- g_file_delete (fixture->tmp_dir, NULL, &local_error);
++ if (fixture->tmp_dir != NULL)
++ g_file_delete (fixture->tmp_dir, NULL, &local_error);
++
+ g_assert_no_error (local_error);
+ g_clear_object (&fixture->tmp_dir);
+ }
+@@ -360,6 +368,10 @@ test_atomic_replace (Fixture *fixture,
+ GError *error = NULL;
+ TestData data;
+
++ /* respect g_test_skip() during setup() */
++ if (g_test_failed ())
++ return;
++
+ data.step = 0;
+ data.events = NULL;
+
+@@ -465,6 +477,10 @@ test_file_changes (Fixture *fixture,
+ GError *error = NULL;
+ TestData data;
+
++ /* respect g_test_skip() during setup() */
++ if (g_test_failed ())
++ return;
++
+ data.step = 0;
+ data.events = NULL;
+
+@@ -582,6 +598,10 @@ test_dir_monitor (Fixture *fixture,
+ GError *error = NULL;
+ TestData data;
+
++ /* respect g_test_skip() during setup() */
++ if (g_test_failed ())
++ return;
++
+ data.step = 0;
+ data.events = NULL;
+
+@@ -679,6 +699,10 @@ test_dir_non_existent (Fixture *fixture,
+ TestData data;
+ GError *error = NULL;
+
++ /* respect g_test_skip() during setup() */
++ if (g_test_failed ())
++ return;
++
+ data.step = 0;
+ data.events = NULL;
+
+@@ -788,6 +812,10 @@ test_cross_dir_moves (Fixture *fixture,
+ GError *error = NULL;
+ TestData data[2];
+
++ /* respect g_test_skip() during setup() */
++ if (g_test_failed ())
++ return;
++
+ data[0].step = 0;
+ data[0].events = NULL;
+
+@@ -957,6 +985,10 @@ test_file_hard_links (Fixture *fixture,
+ GError *error = NULL;
+ TestData data;
+
++ /* respect g_test_skip() during setup() */
++ if (g_test_failed ())
++ return;
++
+ g_test_bug ("755721");
+
+ #ifdef HAVE_LINK
- 70/115: gnu: python-keras: Enable parallel tests., (continued)
- 70/115: gnu: python-keras: Enable parallel tests., guix-commits, 2021/10/15
- 71/115: gnu: dbus-c++: Fix build., guix-commits, 2021/10/15
- 74/115: gnu: python-keras: Skip a flaky test., guix-commits, 2021/10/15
- 77/115: gnu: tzdata: Update to 2021c., guix-commits, 2021/10/15
- 83/115: gnu: curl: Honor #:tests?., guix-commits, 2021/10/15
- 75/115: gnu: glibc: Remove unneeded nscd patching., guix-commits, 2021/10/15
- 78/115: gnu: gtk+-2: Fix ‘builder’ test., guix-commits, 2021/10/15
- 79/115: gnu: diffutils: Fix signal processing., guix-commits, 2021/10/15
- 80/115: gnu: diffutils: Update to 3.8., guix-commits, 2021/10/15
- 84/115: gnu: gcc-5: Fix powerpc64le-linux build, guix-commits, 2021/10/15
- 81/115: gnu: glib: Skip ‘testfilemonitor’ test.,
guix-commits <=
- 86/115: gnu: Add ld-gold-wrapper., guix-commits, 2021/10/15
- 87/115: gnu: gdb: Patch references to /bin/sh and add debug output., guix-commits, 2021/10/15
- 88/115: gnu: gdb: Normalize indentation., guix-commits, 2021/10/15
- 92/115: gnu: rust: Bootstrap rust from 1.39.0 and optimize build time., guix-commits, 2021/10/15
- 72/115: gnu: libnice: Update to 0.1.18-0.47a9633., guix-commits, 2021/10/15
- 85/115: gnu: make-ld-wrapper: Add a LINKER argument., guix-commits, 2021/10/15
- 102/115: gnu: python-pathlib2: Update to 2.3.6., guix-commits, 2021/10/15
- 108/115: gnu: inkscape: Remove the legacy 0.92 version., guix-commits, 2021/10/15
- 100/115: gnu: python-setuptools-scm: Update to 6.3.2, guix-commits, 2021/10/15
- 101/115: gnu: python-pytest-6: Fix version via setuptools-scm., guix-commits, 2021/10/15