Skip to content

Downgrade Pillow pip requirement#2199

Merged
longjon merged 1 commit intoBVLC:masterfrom
lukeyeager:downgrade-pillow
Mar 27, 2015
Merged

Downgrade Pillow pip requirement#2199
longjon merged 1 commit intoBVLC:masterfrom
lukeyeager:downgrade-pillow

Conversation

@lukeyeager
Copy link
Copy Markdown
Contributor

2.7.0 isn't really necessary - 2.3.0 is sufficient. This is the version available on Ubuntu 14.04 via apt-get, and seems to be a reasonable lowest common denominator in general.
http://pillow.readthedocs.org/installation.html#platform-support

See my comment on #2010. I haven't gotten any feedback from @danielhamngren or anyone else about this. As with #2192, the tests pass. And I haven't seen any problems on my end with 2.3.0.

2.7.0 isn't really necessary - 2.3.0 is sufficient. This is the version
available on Ubuntu 14.04 via apt-get, and seems to be a reasonable lowest
common denominator in general.

http://pillow.readthedocs.org/installation.html#old-versions
@shelhamer shelhamer added the JL label Mar 25, 2015
@longjon
Copy link
Copy Markdown
Contributor

longjon commented Mar 27, 2015

Tests pass, and it seems that there was no reason to require 2.7.0, so merging. Thanks!

longjon added a commit that referenced this pull request Mar 27, 2015
@longjon longjon merged commit a2c7cf4 into BVLC:master Mar 27, 2015
@lukeyeager lukeyeager deleted the downgrade-pillow branch March 27, 2015 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants