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

Don't require win_unicode_console on Python 3.6 #9946

Merged
merged 1 commit into from Sep 13, 2016

Conversation

takluyver
Copy link
Member

It looks like the changes @Drekin's win_unicode_console makes have essentially been integrated into Python 3.6, so it will use the Unicode APIs for Windows console access by default. See PEP 528.

So we shouldn't need to load WUC on Python 3.6, and it may even have some strange interactions.

It looks like the changes Drekin's win_unicode_console makes have
essentially been integrated into Python 3.6, so it will use the Unicode
APIs for Windows console access by default. See [PEP
528](https://www.python.org/dev/peps/pep-0528/).

So we shouldn't need to load WUC on Python 3.6, and it may even have
some strange interactions.
@@ -218,7 +218,8 @@ def run(self):
':python_version == "2.7" or python_version == "3.3"': ['pathlib2'],
':sys_platform != "win32"': ['pexpect'],
':sys_platform == "darwin"': ['appnope'],
':sys_platform == "win32"': ['colorama', 'win_unicode_console>=0.5'],
':sys_platform == "win32"': ['colorama'],
':sys_platform == "win32" and python_version < "3.6"': ['win_unicode_console>=0.5'],
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This syntax should be valid as of setuptools 17.1, released June 2015. I'm OK with depending on that.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already have similar ones above, I suspect we already require something similar.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Specifically the < comparison - I believe == and != were supported before other comparsions.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing that since setuptools 17 is more than a year ago, I suspect that most people with setuptools < 17 have setuptools 0.6 + pip 1.5.6, and need to upgrade anyway. I doubt many folks are stuck on setuptools 16.

@minrk minrk merged commit 9cf9dee into ipython:master Sep 13, 2016
@minrk
Copy link
Member

minrk commented Sep 13, 2016

Cool!

@minrk
Copy link
Member

minrk commented Sep 13, 2016

Backport to 5.2?

@takluyver takluyver added this to the 5.2 milestone Sep 13, 2016
@takluyver
Copy link
Member Author

I think it probably makes sense to, yeah. Though do we want to be a bit more conservative with the setuptools version required for 5.x?

@takluyver takluyver deleted the py36-win-unicode branch September 13, 2016 10:25
Carreau added a commit to Carreau/ipython that referenced this pull request Sep 18, 2016
… 3.6

It looks like the changes _Drekin's win_unicode_console makes have essentially been integrated into Python 3.6, so it will use the Unicode APIs for Windows console access by default. See [PEP 528](https://www.python.org/dev/peps/pep-0528/).

So we shouldn't need to load WUC on Python 3.6, and it may even have some strange interactions.
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