Skip to content

Support Python 3.4 #592

Closed
Closed

Description

@aclark4life
No description provided.

Activity

  1. added this to the 2.5.0 milestone on Apr 1, 2014
  2. hugovk commented on Apr 2, 2014

    @hugovk
    Member

    Travis CI are in the process of adding 3.4: travis-ci/travis-ci#1989

    Once it's ready, it can be added to .travis.yml. In fact, it could be added now as an allowed failure.

  3. aclark4life commented on Apr 2, 2014

    @aclark4life
    MemberAuthor

    Thanks @hugovk

  4. cgohlke commented on Apr 2, 2014

    @cgohlke
    Contributor

    Pillow 2.4 passes all tests on CPython 3.4 for Windows.
    The only issue I noticed is that the tcl and tk libraries included with CPython 3.4 are now ending with t, e.g. tcl86t.lib and tk86t.lib, not tcl86.lib and tk86.lib. See https://github.com/python-imaging/Pillow/blob/master/setup.py#L414.

  5. wiredfool commented on Apr 4, 2014

    @wiredfool
    Member

    I'm seeing that Py3.4 is triggering a PendingDeprecationWarning for the imp module, in favor of the importlib module.

  6. hugovk commented on Apr 9, 2014

    @hugovk
    Member

    @wiredfool Where is the imp module used by Pillow? I can't find it anywhere.

  7. Terseus commented on Apr 9, 2014

    @Terseus
    Contributor

    There is a bug in Python 3.4 where the flag -Werror=declaration-after-statement is added for extension modules when it shouldn't, which cause the version of ImageDraw in my PR #610 to fail in compile-time if the workaround described in the Python issue isn't applied.

  8. wiredfool commented on Apr 18, 2014

    @wiredfool
    Member

    @hugovk That's in the windows build, and the deadsnakes ppa for Precise:

    /usr/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)
    

    I'm guessing it's the import calls in Image.py

  9. wiredfool commented on Apr 18, 2014

    @wiredfool
    Member

    Apart from that, it's passing all the current tests for me.

  10. hugovk commented on Apr 30, 2014

    @hugovk
    Member

    Python 3.4 is now available on Travis CI and enabled for Pillow.

    But imp's PendingDeprecationWarning is causing test failures because the test runner doesn't like print output. For example:

    https://travis-ci.org/python-imaging/Pillow/jobs/24087446

  11. hugovk commented on May 8, 2014

    @hugovk
    Member

    The PendingDeprecationWarning is coming from Coverage.py, specifically import coverage on line 348 of tester.py:

    https://github.com/python-imaging/Pillow/blob/master/Tests/tester.py#L348

    Coverage.py, execfile.py, line 3, import imp:

    https://github.com/nedbat/coveragepy/blob/master/coverage/execfile.py#L3

  12. wiredfool commented on May 8, 2014

    @wiredfool
    Member

    We should just ignore that warning then, and revisit as coverage evolves. I'd like to make py3.4 a required passing test.

  13. hugovk commented on May 14, 2014

    @hugovk
    Member

    This issue can be closed.

    Python 3.4 is now supported as a required passing build:
    https://travis-ci.org/python-imaging/Pillow

    I reported the warning to Coverage.py, and will update in another issue as needed.

  14. aclark4life commented on May 14, 2014

    @aclark4life
    MemberAuthor

    Thanks

  15. hugovk commented on May 17, 2014

    @hugovk
    Member

    Update: Ned Batchelder has fixed the imp warning in Coverage.py for Python 3.4:

    Fixed in the commits leading to aa1d5d23ca07

    Not yet released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions