commit:     e3a69b9e068c2e919d14f4a848711919fe1ea7ed
Author:     Marek Szuba <marecki <AT> gentoo <DOT> org>
AuthorDate: Fri Apr 17 17:50:19 2020 +0000
Commit:     Marek Szuba <marecki <AT> gentoo <DOT> org>
CommitDate: Fri Apr 17 17:57:29 2020 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=e3a69b9e

net-p2p/syncthing: work around ppc64 test failure

For some reason one of the Syncthing tests - TestIssue5063 - times out
on our ppc64 buildhost, with 100% reproducibility. Upstream were not
aware of the problem because apparently they have never run the test
suite on such a system, that said their suggestion to adjust the
time-out threshold appears to do the trick. Make it 20 s i.e. twice what
it originally is, hopefully this will prevent the problem from
resurfacing any time soon.

Closes: https://bugs.gentoo.org/717108
See-also: https://github.com/syncthing/syncthing/issues/6539
Signed-off-by: Marek Szuba <marecki <AT> gentoo.org>

 .../files/syncthing-1.3.4-TestIssue5063_timeout.patch  | 18 ++++++++++++++++++
 net-p2p/syncthing/syncthing-1.3.4-r1.ebuild            |  4 ++++
 net-p2p/syncthing/syncthing-1.4.2.ebuild               |  4 ++++
 3 files changed, 26 insertions(+)

diff --git 
a/net-p2p/syncthing/files/syncthing-1.3.4-TestIssue5063_timeout.patch 
b/net-p2p/syncthing/files/syncthing-1.3.4-TestIssue5063_timeout.patch
new file mode 100644
index 00000000000..3148dc8a976
--- /dev/null
+++ b/net-p2p/syncthing/files/syncthing-1.3.4-TestIssue5063_timeout.patch
@@ -0,0 +1,18 @@
+As of early April 2020, running the Syncthing test suite on the Gentoo ppc64
+buildhost fails due to TestIssue5063 timing out. Increase the time-out
+threshold for this test to allow it to pass.
+
+Other arches we currently support (i.e. amd64, arm and x86) shouldn't be
+affected by the change because thay have always passed so far anyway.
+
+--- a/lib/model/model_test.go
++++ b/lib/model/model_test.go
+@@ -914,7 +914,7 @@
+       }()
+       select {
+       case <-finished:
+-      case <-time.After(10 * time.Second):
++      case <-time.After(20 * time.Second):
+               pprof.Lookup("goroutine").WriteTo(os.Stdout, 1)
+               t.Fatal("Timed out before all devices were added")
+       }

diff --git a/net-p2p/syncthing/syncthing-1.3.4-r1.ebuild 
b/net-p2p/syncthing/syncthing-1.3.4-r1.ebuild
index 57a69fe4269..8027187afc5 100644
--- a/net-p2p/syncthing/syncthing-1.3.4-r1.ebuild
+++ b/net-p2p/syncthing/syncthing-1.3.4-r1.ebuild
@@ -292,6 +292,10 @@ RDEPEND="acct-group/syncthing
 
 DOCS=( README.md AUTHORS CONTRIBUTING.md )
 
+PATCHES=(
+       "${FILESDIR}"/${PN}-1.3.4-TestIssue5063_timeout.patch
+)
+
 src_prepare() {
        # Bug #679280
        xdg_environment_reset

diff --git a/net-p2p/syncthing/syncthing-1.4.2.ebuild 
b/net-p2p/syncthing/syncthing-1.4.2.ebuild
index 7b3436014f4..5047b42159a 100644
--- a/net-p2p/syncthing/syncthing-1.4.2.ebuild
+++ b/net-p2p/syncthing/syncthing-1.4.2.ebuild
@@ -306,6 +306,10 @@ RDEPEND="acct-group/syncthing
 
 DOCS=( README.md AUTHORS CONTRIBUTING.md )
 
+PATCHES=(
+       "${FILESDIR}"/${PN}-1.3.4-TestIssue5063_timeout.patch
+)
+
 src_prepare() {
        # Bug #679280
        xdg_environment_reset

Reply via email to