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

Fix output of unicode strings with Python 3 #233

Closed
wants to merge 1 commit into from
Closed

Conversation

sbraz
Copy link

@sbraz sbraz commented Feb 14, 2016

With the current code, Python 3 prints bytes instead of str, which causes the output to be broken (this is especially problematic for --list because the newlines are not interpreted).
I tested my fix with Python 2.4, 2.7 and 3.5. It should not break anything for old versions.

This passes pep8 and pyflakes. I don't have a pypy install to test with but I don't see why it would fail as it's fairly trivial.

@sivel
Copy link
Owner

sivel commented Feb 14, 2016

This is already fixed in another branch that will become the next major release.

@sivel sivel closed this Feb 14, 2016
@sbraz sbraz mentioned this pull request Feb 14, 2016
gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Feb 24, 2016
Upstream will not fix this until the next release
sivel/speedtest-cli#233

Package-Manager: portage-2.2.27
Repository owner locked and limited conversation to collaborators Nov 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants