Skip to content

Compatibility issue with cx_Freeze with Python 2.7 #263

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

Closed
PierreRaybaut opened this issue Nov 30, 2015 · 2 comments
Closed

Compatibility issue with cx_Freeze with Python 2.7 #263

PierreRaybaut opened this issue Nov 30, 2015 · 2 comments

Comments

@PierreRaybaut
Copy link

Currently, jsonschema breaks any attempt to deploy a Python script using cx_Freeze, a common tool for deploying self-consistent Python applications, (not only, but for example) on Windows platforms.

The source of the problem is in jsonschema/compat.py:

try:
    from collections import MutableMapping, Sequence  # noqa
except ImportError:
    from collections.abc import MutableMapping, Sequence  # noqa

When commenting those lines, cx_Freeze works as expected.

The fact is jsonschema is part of the huge dependency list of IPython. IPython is very invasive when installed: even if not needed by your application, deploying with cx_Freeze will take all IPython related modules with it (and jsonschema is among them).
In other words, building the most insignificant Python script will fail only because IPython is installed (which happens a lot) and it depends on jsonschema.

See PR #264

@Julian
Copy link
Member

Julian commented Dec 1, 2015

For anyone finding this: see the discussion on #264, which was a proposed fix (and a probably working one I assume :), for what seems like an upstream issue, but it'd need a test to be mergeable.

@Julian
Copy link
Member

Julian commented Feb 18, 2016

It seems like the upstream issue was fixed, so I'm going to close this as "unsupported, but willing to support if the above is addressed."

@Julian Julian closed this as completed Feb 18, 2016
Julian added a commit that referenced this issue Jun 17, 2019
e68ff511 Straight up copy Draft 7 tests to Draft 2019-06.
1bd999ac Merge pull request #263 from juxt/master
a43c1d17 Add second Clojure implementation
eadeacb0 Merge pull request #262 from leadpony/fix-ref
3aa8d8fc Fix $ref value as properly percent-encoded.
8feaea4f The test suite schema is draft 4, not draft3 anymore.

git-subtree-dir: json
git-subtree-split: e68ff51148e0acd3476ea6937689552d3dc34a3e
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

No branches or pull requests

2 participants