[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#861578: marked as done (unblock: cairocffi/0.7.2-2)



Your message dated Mon, 01 May 2017 06:16:00 +0000
with message-id <7d74c141-4289-bc20-211d-484313941b91@thykier.net>
and subject line Re: Bug#861578: unblock: cairocffi/0.7.2-2
has caused the Debian Bug report #861578,
regarding unblock: cairocffi/0.7.2-2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
861578: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=861578
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package cairocffi

* it drops all the depencencies on xcffib as that pkg FTBFS
* it updates the VCS url, pointing to the HTTPS version (I consider this
  information begin updated and a valuable fix for stretch lifetime)
* if fixes a random FTBFS in the test_xcb.py tests. Now, test_xcb.py is disabled
  because we removed xcffib from b-d (so for this upload is a no-op) but once
  we'll be able to add it back, it's a valuable contribution (and also it was
  already in git so i was somewhat lazy and left it in the upload)

Source debdiff is attached

unblock cairocffi/0.7.2-2

-- System Information:
Debian Release: stretch/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64
 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.2.0-1-amd64 (SMP w/8 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
diff -Nru cairocffi-0.7.2/debian/changelog cairocffi-0.7.2/debian/changelog
--- cairocffi-0.7.2/debian/changelog	2015-12-10 13:02:02.000000000 -0500
+++ cairocffi-0.7.2/debian/changelog	2017-04-30 20:32:16.000000000 -0400
@@ -1,3 +1,20 @@
+cairocffi (0.7.2-2) unstable; urgency=medium
+
+  [ Ondřej Nový ]
+  * Fixed VCS URL (https)
+
+  [ Stefano Rivera ]
+  * Fix XCB tests (Closes: #828929)
+
+  [ Christoph Reiter ]
+  * Drop Build-Depends and Recommends on xcffib and skip xcb tests
+    (Closes: #861175)
+
+  [ Sandro Tosi ]
+  * Team upload.
+
+ -- Sandro Tosi <morph@debian.org>  Sun, 30 Apr 2017 20:32:16 -0400
+
 cairocffi (0.7.2-1) unstable; urgency=medium
 
   * New upstream release
diff -Nru cairocffi-0.7.2/debian/control cairocffi-0.7.2/debian/control
--- cairocffi-0.7.2/debian/control	2015-12-10 13:02:02.000000000 -0500
+++ cairocffi-0.7.2/debian/control	2017-04-30 20:32:16.000000000 -0400
@@ -10,7 +10,6 @@
 		python3-all,  
 		python3-setuptools,
 		python-cffi (>= 1.1.0), python3-cffi (>= 1.1.0),
-		python-xcffib (>= 0.3.2), python3-xcffib (>= 0.3.2),
 		python-sphinx | python3-sphinx, 
 		python-pytest, python3-pytest,
 		xvfb, xauth,
@@ -22,13 +21,12 @@
 X-Python3-Version: >= 3.2
 Standards-Version: 3.9.6
 Homepage: https://pythonhosted.org/cairocffi/
-Vcs-Git: git://anonscm.debian.org/python-modules/packages/cairocffi.git
+Vcs-Git: https://anonscm.debian.org/git/python-modules/packages/cairocffi.git
 Vcs-Browser: https://anonscm.debian.org/cgit/python-modules/packages/cairocffi.git
 
 Package: python-cairocffi
 Architecture: all
 Depends: ${misc:Depends}, ${python:Depends}, python-cffi, libcairo2, libgdk-pixbuf2.0-0
-Recommends: python-xcffib (>= 0.3.2)
 Suggests: python-cairocffi-doc
 Description: cffi-based cairo bindings for Python 
  cairocffi is a CFFI-based drop-in replacement for Pycairo,
@@ -39,7 +37,6 @@
 Package: python3-cairocffi
 Architecture: all
 Depends: ${misc:Depends}, ${python3:Depends}, python3-cffi, libcairo2, libgdk-pixbuf2.0-0
-Recommends: python3-xcffib (>= 0.3.2)
 Suggests: python-cairocffi-doc
 Description: cffi-based cairo bindings for Python (Python3)
  cairocffi is a CFFI-based drop-in replacement for Pycairo,
diff -Nru cairocffi-0.7.2/debian/.git-dpm cairocffi-0.7.2/debian/.git-dpm
--- cairocffi-0.7.2/debian/.git-dpm	2015-12-10 13:02:02.000000000 -0500
+++ cairocffi-0.7.2/debian/.git-dpm	2017-04-30 20:32:16.000000000 -0400
@@ -1,6 +1,6 @@
 # see git-dpm(1) from git-dpm package
-c60e548d413fe20495a53d6e5059d3805cb8f00b
-c60e548d413fe20495a53d6e5059d3805cb8f00b
+72067bb5a6d92564284f5a6a94369f24cd4405bb
+72067bb5a6d92564284f5a6a94369f24cd4405bb
 442c25e12a2070d915ac16d4daba19575babe9fa
 442c25e12a2070d915ac16d4daba19575babe9fa
 cairocffi_0.7.2.orig.tar.gz
diff -Nru cairocffi-0.7.2/debian/patches/fix-xcb-tests.patch cairocffi-0.7.2/debian/patches/fix-xcb-tests.patch
--- cairocffi-0.7.2/debian/patches/fix-xcb-tests.patch	1969-12-31 19:00:00.000000000 -0500
+++ cairocffi-0.7.2/debian/patches/fix-xcb-tests.patch	2017-04-30 20:32:16.000000000 -0400
@@ -0,0 +1,66 @@
+From 72067bb5a6d92564284f5a6a94369f24cd4405bb Mon Sep 17 00:00:00 2001
+From: Sean Vig <sean.v.775@gmail.com>
+Date: Tue, 6 Sep 2016 22:38:14 -0700
+Subject: Fix XCB tests
+
+Make sure XCB objects are free'd, and cffi objects set to None so they
+are gc'd and destroyed before the connection is disconnected.
+
+Origin: https://github.com/Kozea/cairocffi/pull/68
+Bug-Debian: https://bugs.debian.org/828929
+
+Patch-Name: fix-xcb-tests.patch
+---
+ cairocffi/test_xcb.py | 18 ++++++++++++++++--
+ 1 file changed, 16 insertions(+), 2 deletions(-)
+
+diff --git a/cairocffi/test_xcb.py b/cairocffi/test_xcb.py
+index c8a6e70..1aa5123 100644
+--- a/cairocffi/test_xcb.py
++++ b/cairocffi/test_xcb.py
+@@ -87,6 +87,10 @@ def create_pixmap(conn, wid, width, height):
+     return pixmap
+ 
+ 
++def remove_pixmap(conn, pixmap):
++    conn.core.FreePixmap(pixmap)
++
++
+ def create_gc(conn):
+     """Creates a simple graphics context"""
+     gc = conn.generate_id()
+@@ -104,8 +108,10 @@ def create_gc(conn):
+     return gc
+ 
+ 
+-@pytest.mark.xfail(cairo_version() < 11200,
+-                   reason="Cairo version too low")
++def remove_gc(conn, gc):
++    conn.core.FreeGC(gc)
++
++
+ def test_xcb_pixmap(xcb_conn):
+     width = 10
+     height = 10
+@@ -149,6 +155,11 @@ def test_xcb_pixmap(xcb_conn):
+         width, height
+     )
+ 
++    ctx = None
++    surface = None
++    remove_gc(xcb_conn, gc)
++    remove_pixmap(xcb_conn, pixmap)
++
+     # flush the connection, make sure no errors were thrown
+     xcb_conn.flush()
+     while event:
+@@ -220,6 +231,9 @@ def test_xcb_window(xcb_conn):
+     ctx.set_source_rgb(1, 1, 1)
+     ctx.paint()
+ 
++    ctx = None
++    surface = None
++
+     # flush the connection, make sure no errors were thrown
+     xcb_conn.flush()
+     while event:
diff -Nru cairocffi-0.7.2/debian/patches/series cairocffi-0.7.2/debian/patches/series
--- cairocffi-0.7.2/debian/patches/series	2015-12-10 13:02:02.000000000 -0500
+++ cairocffi-0.7.2/debian/patches/series	2017-04-30 20:32:16.000000000 -0400
@@ -1,2 +1,3 @@
 fix_import_sphinxdoc.patch
 use_local_objects.inv.patch
+fix-xcb-tests.patch
diff -Nru cairocffi-0.7.2/debian/rules cairocffi-0.7.2/debian/rules
--- cairocffi-0.7.2/debian/rules	2015-12-10 13:02:02.000000000 -0500
+++ cairocffi-0.7.2/debian/rules	2017-04-30 20:32:16.000000000 -0400
@@ -5,6 +5,7 @@
 export PYBUILD_NAME=cairocffi
 
 export PYBUILD_TEST_PYTEST=1
+export PYBUILD_TEST_ARGS=-k "not test_xcb.py"
 export PYBUILD_AFTER_TEST=rm -rf {build_dir}/*/__pycache__
 
 %:

--- End Message ---
--- Begin Message ---
Sandro Tosi:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please unblock package cairocffi
> 
> * it drops all the depencencies on xcffib as that pkg FTBFS
> * it updates the VCS url, pointing to the HTTPS version (I consider this
>   information begin updated and a valuable fix for stretch lifetime)
> * if fixes a random FTBFS in the test_xcb.py tests. Now, test_xcb.py is disabled
>   because we removed xcffib from b-d (so for this upload is a no-op) but once
>   we'll be able to add it back, it's a valuable contribution (and also it was
>   already in git so i was somewhat lazy and left it in the upload)
> 
> Source debdiff is attached
> 
> unblock cairocffi/0.7.2-2
> 
> [...]

Thanks for taking care of this stretch-is-blocker issue! :)

Unblocked, thanks.
~Niels

--- End Message ---

Reply to: