Bug 1290632

Summary: xception: tostring() has been removed. Please call tobytes() instead.
Product: [Fedora] Fedora Reporter: David Shea <dshea>
Component: comixAssignee: Mamoru TASAKA <mtasaka>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 23CC: mtasaka
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: comix-4.0.4-9.fc23 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-25 00:27:29 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description David Shea 2015-12-11 00:45:28 UTC
Description of problem:

There is a traceback due a change in PIL that is causing the thumbnails to not be displayed. Here's the whole thing:

Traceback (most recent call last):
  File "/usr/share/comix/src/thumbbar.py", line 126, in _load
    _add_page_number(pixbuf, i)
  File "/usr/share/comix/src/thumbbar.py", line 201, in _add_page_number
    num_pixbuf = image.pil_to_pixbuf(im)
  File "/usr/share/comix/src/image.py", line 172, in pil_to_pixbuf
    imagestr = image.tostring()
  File "/usr/lib64/python2.7/site-packages/PIL/Image.py", line 686, in tostring
    "Please call tobytes() instead.")
Exception: tostring() has been removed. Please call tobytes() instead.



Version-Release number of selected component (if applicable):
comix-4.0.4-8.fc23.1.noarch


Seems to work fine if you just patch image.py with:

diff -purN comix-4.0.4.orig/src/image.py comix-4.0.4/src/image.py
--- comix-4.0.4.orig/src/image.py	2015-12-10 19:42:53.740805488 -0500
+++ comix-4.0.4/src/image.py	2015-12-10 19:43:17.793817018 -0500
@@ -169,7 +169,7 @@ def get_most_common_edge_colour(pixbuf):
 
 def pil_to_pixbuf(image):
     """Return a pixbuf created from the PIL <image>."""
-    imagestr = image.tostring()
+    imagestr = image.tobytes()
     IS_RGBA = image.mode == 'RGBA'
     return gtk.gdk.pixbuf_new_from_data(imagestr, gtk.gdk.COLORSPACE_RGB,
         IS_RGBA, 8, image.size[0], image.size[1],

Comment 1 Mamoru TASAKA 2015-12-11 00:54:57 UTC
*When* did this change occur? After F-23 alpha? If so, I think python-pillow change must be reverted.

Comment 2 Fedora Update System 2015-12-11 06:54:13 UTC
comix-4.0.4-9.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-f0fd9c8ee5

Comment 3 Mamoru TASAKA 2015-12-11 06:55:37 UTC
Anyway patch applied, thank you.

Comment 4 Fedora Update System 2015-12-12 03:23:30 UTC
comix-4.0.4-9.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update comix'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-f0fd9c8ee5

Comment 5 Fedora Update System 2015-12-25 00:27:27 UTC
comix-4.0.4-9.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.