Opened 10 years ago
Closed 2 years ago
#651 closed enhancement (fixed)
Switch to Pillow?
|Reported by:||Christopher Allan Webber||Owned by:|
This has been mentioned by willkg in several previous bugs: #392 #57 (and one where someone mentioned it as a fix: #412) but maybe we should switch from PIL to Pillow; PIL seems to be better maintained: https://pypi.python.org/pypi/Pillow/
However, we shouldn't just make this switch without considering the implications. For example, we are currently shooting for Debian packaging, but Pillow is not packaged in Debian. Pillow is apparently backwards compatible though?
Change History (10)
comment:1 by , 10 years ago
comment:2 by , 10 years ago
Debian has it in the experimental version of python-imaging, however there is a bug that is stopping it from continuing. While the module is backward compatible, the import statement is not. As I say in the bug report, this is deliberate and necessary to get setuptools working.
comment:3 by , 7 years ago
CentOS 7 now has pillow, since we dropped python 2.6 support there is only the x86_64 bit version of CentOS 6 which doesn't have pillow. I suggest we go forth and switch, it seems everyone else is.
comment:4 by , 7 years ago
I don't think this is an OS-specific issue. Pillow is available on PyPI, so regardless of what packages are available for a specific distro, one should be able to install Pillow in their mediagoblin environment using
easy_install. We could add this to setup.py.
comment:5 by , 7 years ago
We can't add PIL or Pillow to setup.py without requiring the user also have gcc and other build utilities installed. Pillow (like PIL) requires some compilation (it's a mixture of C and python IIRC). We currently don't want to ask people to have those installed so we rely on the package manager to provide them.
comment:6 by , 7 years ago
Oh, I see. Understood.
comment:7 by , 7 years ago
So I think at this point, this is just a documentation change, right?
commit d0e9f843e23196f790b640327bef3718e74b0347 Author: Alon Levy <firstname.lastname@example.org> Date: Mon Apr 8 11:19:56 2013 +0300 PIL: Support systems with Pillow and without Fixes for systems with Pillow, but leaves a "try: except ImportError" to support anything that doesn't have a PIL top level import. Signed-off-by: Alon Levy <email@example.com>
It looks like we just have to switch the documentation over to recommending Pillow and dropping references to PIL. We could then also drop support for PIL specifically in the imports; no need to support both anymore...
comment:8 by , 7 years ago
Yep, I believe so. I actually use pillow on all my dev servers already ;)
comment:9 by , 7 years ago
Netbulae notes in #911 "configure: error: Python Imaging Library is required" that Fedora 20 doesn't have
python-imaging and instead has
python-pillow. Keeping that ticket open, since it could probably be solved simply by updating documentation.
comment:10 by , 2 years ago
|Status:||new → closed|
Already switched to Pillow.
Tsyesika points out that debian, redhat, et c seems to still be using PIL, so switching entirely over this might be an issue there.