Bug 896296

Summary: Port polybori's use of PIL to support Pillow for F19
Product: [Fedora] Fedora Reporter: Toshio Ernie Kuratomi <a.badger>
Component: polyboriAssignee: Jerry James <loganjerry>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: loganjerry
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-01-21 23:26:04 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:
Bug Depends On:    
Bug Blocks: 894484    
Attachments:
Description Flags
Patch to use the form of import of PIL that is pillow compatible none

Description Toshio Ernie Kuratomi 2013-01-17 00:28:08 UTC
Created attachment 679942 [details]
Patch to use the form of import of PIL that is pillow compatible

Description of problem:

Fedora 19 is going to stop shipping python-imaging (PIL) and start shipping python-pillow instead.  Rationale can be found on the feature page: https://fedoraproject.org/wiki/Features/Pillow .  pillow should be compatible with PIL at the code level but the import statement changes slightly.

PIL supports both of these:

import Image

from PIL import Image

Pillow only supports the latter form.

I grepped through the polybori source and found some occurrences of "import Image".  I'll attach a patch that updates these to the new form of import.

Comment 1 Jerry James 2013-01-17 04:34:41 UTC
Thanks for the patch.  Will a BR of python-imaging-devel and an R of python-imaging still be correct, or will they need to be changed too?

Comment 2 Toshio Ernie Kuratomi 2013-01-17 06:23:38 UTC
The plan is to have python-pillow Provide python-imaging{-devel,-tk,etc} since the API is the same except for dropping the old method of importing.

You can update the package to use python-pillow and python-pillow-devel once those packages have replaced python-imaging in the repository if you like but it shouldn't be necessary for at least the next few releases.

Comment 3 Jerry James 2013-01-21 23:26:04 UTC
I just rebuilt polybori in Rawhide with this patch, and forwarded the patch upstream.  Thanks, Toshio.