From 3c87f0a304dca5df9c8e39367dc6572726a6e4fc Mon Sep 17 00:00:00 2001
From: Benjamin A. Beasley <code@musicinmybrain.net>
Date: Oct 16 2023 20:42:18 +0000
Subject: F40+: Exclude s390x; <=F39: Skip tests on s390x


---

diff --git a/python-pypet.spec b/python-pypet.spec
index 5455d1e..90f7390 100644
--- a/python-pypet.spec
+++ b/python-pypet.spec
@@ -1,4 +1,13 @@
-%bcond tests 1
+# We shouldn’t *add* ExcludeArch in a stable release, so we must skip the tests
+# instead. We can remove this after F39 goes end-of-life, or when the spec
+# files for older branches diverge from those in F40+.
+%ifarch s390x
+%if 0%{?fc38} || 0%{?fc39}
+%global tests_default_off 1
+%endif
+%endif
+
+%bcond tests 0%{?!tests_default_off:1}
 # Run examples as additional tests?
 %bcond test_examples 1
 
@@ -56,13 +65,25 @@ Patch:          %{url}/pull/71.patch
 # architecture-dependent failures. However, there is no compiled code in the
 # package.
 %global debug_package %{nil}
-%if !0%{?fc38}
+# ==== Exclude i686 ====
+#
 # https://fedoraproject.org/wiki/Changes/EncourageI686LeafRemoval
 #
 # Also, mandatory dependency python-tables dropped i686 support in F39 because
 # python-blosc2 does not support it:
 # https://src.fedoraproject.org/rpms/python-tables/c/ee27ac0dd4352ee415ad5089aee76c50f4bd2028
+#
+# ==== Exclude s390x ====
+#
+# Many python-pypet tests fail on s390x due to apparent endian issues in
+# pandas.HDFStore
+# https://bugzilla.redhat.com/show_bug.cgi?id=2244500
 ExcludeArch:    %{ix86}
+# We shouldn’t *add* ExcludeArch in a stable release. We can remove the
+# conditional after F39 goes end-of-life, or when the spec files for older
+# branches diverge from those in F40+.
+%if !0%{?fc38} && !0%{?fc39}
+ExcludeArch:    s390x
 %endif
 
 %description %_description