unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
From: Josh Holland <josh@inv.alid.pw>
To: 36675@debbugs.gnu.org
Cc: Josh Holland <josh@inv.alid.pw>
Subject: bug#36675: [PATCH] gnu: kodi: Disable intermittently failing test.
Date: Tue,  3 Dec 2019 13:52:14 +0000	[thread overview]
Message-ID: <20191203135214.1871840-1-josh@inv.alid.pw> (raw)
In-Reply-To: <87wogj3xzo.fsf@inv.alid.pw>

As reported in <https://issues.guix.gnu.org/issue/36675>, occasionally the
TestMassEvent.General test in kodi crashes with a mutex error.

* gnu/packages/patches/kodi-skip-testmasseventgeneral.patch: New file.
* gnu/packages/kodi.scm (kodi)[source]: Use it.
* gnu/local.mk: (dist_patch_DATA): Add it.
---
I have tested building the kodi-wayland package with this patch, and
it successfully builds and the tests pass.

 gnu/local.mk                                  |  1 +
 gnu/packages/kodi.scm                         |  3 +-
 .../kodi-skip-testmasseventgeneral.patch      | 59 +++++++++++++++++++
 3 files changed, 62 insertions(+), 1 deletion(-)
 create mode 100644 gnu/packages/patches/kodi-skip-testmasseventgeneral.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index e2d922373e..4fc690c785 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1036,6 +1036,7 @@ dist_patch_DATA =						\
   %D%/packages/patches/kodi-increase-test-timeout.patch		\
   %D%/packages/patches/kodi-set-libcurl-ssl-parameters.patch		\
   %D%/packages/patches/kodi-skip-test-449.patch		\
+  %D%/packages/patches/kodi-skip-testmasseventgeneral.patch     \
   %D%/packages/patches/laby-make-install.patch			\
   %D%/packages/patches/lcalc-default-parameters-1.patch		\
   %D%/packages/patches/lcalc-default-parameters-2.patch		\
diff --git a/gnu/packages/kodi.scm b/gnu/packages/kodi.scm
index 59ea11f812..04d879bd17 100644
--- a/gnu/packages/kodi.scm
+++ b/gnu/packages/kodi.scm
@@ -281,7 +281,8 @@ alternatives. In compilers, this can reduce the cascade of secondary errors.")
                 "1m0295czxabdcqyqf5m94av9d88pzhnzjvyfs1q07xqq82h313p7"))
               (patches (search-patches "kodi-skip-test-449.patch"
                                        "kodi-increase-test-timeout.patch"
-                                       "kodi-set-libcurl-ssl-parameters.patch"))
+                                       "kodi-set-libcurl-ssl-parameters.patch"
+                                       "kodi-skip-testmasseventgeneral.patch"))
               (snippet
                '(begin
                   (use-modules (guix build utils))
diff --git a/gnu/packages/patches/kodi-skip-testmasseventgeneral.patch b/gnu/packages/patches/kodi-skip-testmasseventgeneral.patch
new file mode 100644
index 0000000000..89fe60b598
--- /dev/null
+++ b/gnu/packages/patches/kodi-skip-testmasseventgeneral.patch
@@ -0,0 +1,59 @@
+This test has been intermittently failing since version 18.2 (see Guix
+bug 36675).
+
+438/521 Test #438: TestMassEvent.General ....................................................Child aborted***Exception:   0.82 sec
+Note: Google Test filter = TestMassEvent.General
+[==========] Running 1 test from 1 test case.
+[----------] Global test environment set-up.
+[----------] 1 test from TestMassEvent
+[ RUN      ] TestMassEvent.General
+[       OK ] TestMassEvent.General (105 ms)
+[----------] 1 test from TestMassEvent (105 ms total)
+
+[----------] Global test environment tear-down
+[==========] 1 test from 1 test case ran. (142 ms total)
+[  PASSED  ] 1 test.
+kodi-test: ../nptl/pthread_mutex_lock.c:434: __pthread_mutex_lock_full: Assertion `robust || (oldval & FUTEX_OWNER_DIED) == 0' failed.
+
+---
+ xbmc/threads/test/TestEvent.cpp | 19 +++++++++----------
+ 1 file changed, 9 insertions(+), 10 deletions(-)
+
+diff --git a/xbmc/threads/test/TestEvent.cpp b/xbmc/threads/test/TestEvent.cpp
+index 40e644c0ed..8fe329caa4 100644
+--- a/xbmc/threads/test/TestEvent.cpp
++++ b/xbmc/threads/test/TestEvent.cpp
+@@ -592,17 +592,17 @@ template <class W> void RunMassEventTest(std::vector<std::shared_ptr<W>>& m, boo
+ }
+
+
+-TEST(TestMassEvent, General)
+-{
+-  g_event = new CEvent();
++// TEST(TestMassEvent, General)
++// {
++//   g_event = new CEvent();
+
+-  std::vector<std::shared_ptr<mass_waiter>> m(NUMTHREADS);
+-  for(size_t i=0; i<NUMTHREADS; i++)
+-    m[i].reset(new mass_waiter());
++//   std::vector<std::shared_ptr<mass_waiter>> m(NUMTHREADS);
++//   for(size_t i=0; i<NUMTHREADS; i++)
++//     m[i].reset(new mass_waiter());
+
+-  RunMassEventTest(m,true);
+-  delete g_event;
+-}
++//   RunMassEventTest(m,true);
++//   delete g_event;
++// }
+
+ TEST(TestMassEvent, Polling)
+ {
+@@ -615,4 +615,3 @@ TEST(TestMassEvent, Polling)
+   RunMassEventTest(m,false);
+   delete g_event;
+ }
+-
+--
+2.24.0
--
2.24.0

  parent reply	other threads:[~2019-12-03 13:53 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-15 19:21 bug#36675: kodi 18.2 fails to build Josh Holland
     [not found] ` <handler.36675.B.156321850120221.ack@debbugs.gnu.org>
2019-07-15 21:38   ` bug#36675: Acknowledgement (kodi 18.2 fails to build) Josh Holland
2021-01-11 13:36     ` bug#36675: kodi 18.2 fails to build Maxim Cournoyer
2019-07-15 22:38 ` bug#36675: (no subject) Amirouche
2019-07-30  9:57 ` bug#36675: Same build failure with kodi 18.3 Josh Holland
2019-07-30 10:20   ` Josh Holland
2019-07-30 14:15     ` Ricardo Wurmus
2019-12-03 13:52 ` Josh Holland [this message]
2020-03-21 20:28   ` bug#36675: [PATCH] gnu: kodi: Disable intermittently failing test Marius Bakke

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191203135214.1871840-1-josh@inv.alid.pw \
    --to=josh@inv.alid.pw \
    --cc=36675@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).