[pygobject3] Fix GBytes test (gnome#690837)
Dan Horák
sharkcz at fedoraproject.org
Fri Dec 28 21:43:13 UTC 2012
commit 4382f556f556d6ea92d8035715c89bee7da0abbe
Author: Dan Horák <dan at danny.cz>
Date: Fri Dec 28 22:43:08 2012 +0100
Fix GBytes test (gnome#690837)
pygobject-3.7.3-test-gbytes-compare.patch | 34 +++++++++++++++++++++++++++++
pygobject3.spec | 10 +++++++-
2 files changed, 43 insertions(+), 1 deletions(-)
---
diff --git a/pygobject-3.7.3-test-gbytes-compare.patch b/pygobject-3.7.3-test-gbytes-compare.patch
new file mode 100644
index 0000000..afe93ea
--- /dev/null
+++ b/pygobject-3.7.3-test-gbytes-compare.patch
@@ -0,0 +1,34 @@
+From 05711e815f5dbb840f10c9f42defb748116617c0 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Dan=20Hor=C3=A1k?= <dan at danny.cz>
+Date: Fri, 28 Dec 2012 22:12:32 +0100
+Subject: [PATCH] test for GBytes.compare must match definition
+
+The result of the compare method is defined as equal, less than or greater than zero
+and the test must match to that. The underlaying memcmp() function can return other
+values than -1, 0 and 1. For example on architectures where it is implemented directly
+via a CPU instruction like on s390(x) where I can see -2 as a result instead of the
+"expected" -1.
+---
+ tests/test_gi.py | 6 +++---
+ 1 files changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/tests/test_gi.py b/tests/test_gi.py
+index bb90b8a..8cb3496 100644
+--- a/tests/test_gi.py
++++ b/tests/test_gi.py
+@@ -1053,9 +1053,9 @@ class TestGBytes(unittest.TestCase):
+ self.assertFalse(a1.equal(b))
+ self.assertFalse(b.equal(a2))
+
+- self.assertEqual(0, a1.compare(a2))
+- self.assertEqual(1, a1.compare(b))
+- self.assertEqual(-1, b.compare(a1))
++ self.assertTrue(a1.compare(a2) == 0)
++ self.assertTrue(a1.compare(b) > 0)
++ self.assertTrue(b.compare(a1) < 0)
+
+
+ class TestGByteArray(unittest.TestCase):
+--
+1.7.7.6
+
diff --git a/pygobject3.spec b/pygobject3.spec
index 0c66e45..304900f 100644
--- a/pygobject3.spec
+++ b/pygobject3.spec
@@ -22,7 +22,7 @@
Name: pygobject3
Version: 3.7.3
-Release: 1%{?dist}
+Release: 2%{?dist}
License: LGPLv2+ and MIT
Group: Development/Languages
Summary: Python 2 bindings for GObject Introspection
@@ -69,6 +69,10 @@ Patch2: pygobject-3.3.4-known-failures.patch
# Not yet sent upstream:
Patch3: test-list-marshalling.patch
+# Fix GBytes.compare test
+# https://bugzilla.gnome.org/show_bug.cgi?id=690837
+Patch4: pygobject-3.7.3-test-gbytes-compare.patch
+
### Build Dependencies ###
BuildRequires: chrpath
@@ -150,6 +154,7 @@ for use in Python 3 programs.
%patch1 -p1 -b .ignore-more-pep8-errors
%patch2 -p1 -b .known-failures
%patch3 -p1 -b .test-list-marshalling
+%patch4 -p1 -b .test-gbytes-compare
%if 0%{?with_python3}
rm -rf %{py3dir}
@@ -263,6 +268,9 @@ xvfb-run make DESTDIR=$RPM_BUILD_ROOT check %{verbosity}
%endif # with_python3
%changelog
+* Fri Dec 28 2012 Dan Horák <dan[at]danny.cz> - 3.7.3-2
+- Fix GBytes test (gnome#690837)
+
* Thu Dec 20 2012 Kalev Lember <kalevlember at gmail.com> - 3.7.3-1
- Update to 3.7.3
- Drop upstreamed patches; rebase the ignore-more-pep8-errors patch
More information about the scm-commits
mailing list