Bug 895214

Summary: Update wavextract import of PIL for F19 switch to Pillow
Product: [Fedora] Fedora Reporter: Stephen Gallagher <sgallagh>
Component: wavextractAssignee: Gwyn Ciesla <gwync>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: rawhideCC: gwync
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-01-15 18:15:40 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
Change imports of PIL Library so they'll work with pillow none

Description Stephen Gallagher 2013-01-14 20:51:37 UTC
Created attachment 678426 [details]
Change imports of PIL Library so they'll work with pillow

Description of problem:

Fedora 19 is switching from using python-imaging (PIL) to python-pillow.  This is mostly backwards compatible but importing the python-imaging relevant libraries is undergoing a change.  In PIL, the libraries could either be imported like this:

import Image

or like this:

from PIL import Image

In python-pillow (and thus, Fedora19+), only the latter will work.  I've grepped the sources of wavextract and found a few import statements that need to be updated.  Patch attached to fix those.  Patch should be upstreamable as PIL supports both the old and new forms of the import.

Version-Release number of selected component (if applicable):

wavextract-1.0.0-8

Comment 1 Gwyn Ciesla 2013-01-15 16:27:05 UTC
Should this require python-imaging still, or python-pillow?

Comment 2 Gwyn Ciesla 2013-01-15 18:15:40 UTC
Updated, using python-imaging.

Comment 3 Stephen Gallagher 2013-01-16 13:33:32 UTC
Sorry for the late response. Python-imaging is fine, since it's a drop-in replacement.