Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python 3.4 is live on Travis CI #641

Merged
merged 3 commits into from Apr 30, 2014
Merged

Conversation

@hugovk
Copy link
Member Author

hugovk commented Apr 30, 2014

Build errored due to problems on Travis CI.

I restarted this older build that has no 3.4. It was previously all green, but now errors on all but 2.7 and 3.2.
https://travis-ci.org/hugovk/Pillow/builds/23292576

I've reported it to Travis CI: travis-ci/travis-ci#2219

… for now, as a PendingDeprecationWarning is causing failures; this is a problem with the test runner not liking any print output.
@hugovk
Copy link
Member Author

hugovk commented Apr 30, 2014

Problem was with these lines in .travis.yml:

# for python-qt4
virtualenv:
  system_site_packages: true

Removing them allows the tests to run.

I put 3.4 as an allowed failure because the homemade test runner chokes on the print output:

running test_000_sanity ...
/home/travis/virtualenv/python3.4.0/lib/python3.4/imp.py:32: PendingDeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  PendingDeprecationWarning)

(cc #592)

aclark4life added a commit that referenced this pull request Apr 30, 2014
Python 3.4 is live on Travis CI
@aclark4life aclark4life merged commit 73eafbb into python-pillow:master Apr 30, 2014
@aclark4life
Copy link
Member

Nice thanks

@hugovk hugovk deleted the patch-1 branch April 30, 2014 09:39
@hugovk
Copy link
Member Author

hugovk commented May 1, 2014

For reference, here's feedback from Travis about system_site_packages. Summary: it's only available for 2.7 and 3.2 and "you can test with and without system site packages for the same Python version".

travis-ci/travis-ci#2219 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants