Skip to content

Python3 compatibility#3

Open
sboeser wants to merge 3 commits into
gravitino:masterfrom
sboeser:master
Open

Python3 compatibility#3
sboeser wants to merge 3 commits into
gravitino:masterfrom
sboeser:master

Conversation

@sboeser
Copy link
Copy Markdown

@sboeser sboeser commented Jun 2, 2016

Tested on Python-3.4.3 and 2.7.10 on OSX.

  • Python3-compatible use of StringIO, urllib and print
  • replaced hcluster by dedupe-hcluster which is Py3 compatible

@moschlar
Copy link
Copy Markdown
Collaborator

moschlar commented Jun 2, 2016

Your changes seem to make the project Python3-only - which is kind of bad, since it is inherently used in SAUCE which is Python2-only...
Could you maybe try using something like https://pypi.python.org/pypi/six to make it compatible with both versions?

@sboeser
Copy link
Copy Markdown
Author

sboeser commented Jun 3, 2016

Hm - I thought I had taken care of that. What exactly doesn't work in Python2.X? I had only tested on 2.7.10... - how far back do you want to support?

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.

2 participants