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

Error after latest update #50

Closed
erkiha opened this issue Mar 11, 2014 · 8 comments
Closed

Error after latest update #50

erkiha opened this issue Mar 11, 2014 · 8 comments

Comments

@erkiha
Copy link

erkiha commented Mar 11, 2014

erki@siil:~$ sudo pip install mps-youtube --upgrade
Downloading/unpacking mps-youtube from https://pypi.python.org/packages/source/m/mps-youtube/mps-youtube-0.01.39.tar.gz#md5=22f0a605eb64261f3eb5916b691db3a4
Downloading mps-youtube-0.01.39.tar.gz
Running setup.py egg_info for package mps-youtube

Requirement already up-to-date: Pafy in /usr/local/lib/python2.7/dist-packages (from mps-youtube)
Installing collected packages: mps-youtube
Found existing installation: mps-youtube 0.01.38
Uninstalling mps-youtube:
Successfully uninstalled mps-youtube
Running setup.py install for mps-youtube
changing mode of build/scripts-2.7/mpsyt from 644 to 755

changing mode of /usr/local/bin/mpsyt to 755

Successfully installed mps-youtube
Cleaning up...
erki@siil:$ mpsyt
Traceback (most recent call last):
File "/usr/local/bin/mpsyt", line 297, in
class Config(object):
File "/usr/local/bin/mpsyt", line 308, in Config
DDIR = get_default_ddir()
File "/usr/local/bin/mpsyt", line 133, in get_default_ddir
defn = [x for x in lines if x.startswith("XDG_DOWNLOAD_DIR")]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 24: ordinal not in range(128)
erki@siil:
$ mpsyt
Traceback (most recent call last):
File "/usr/local/bin/mpsyt", line 297, in
class Config(object):
File "/usr/local/bin/mpsyt", line 308, in Config
DDIR = get_default_ddir()
File "/usr/local/bin/mpsyt", line 133, in get_default_ddir
defn = [x for x in lines if x.startswith("XDG_DOWNLOAD_DIR")]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 24: ordinal not in range(128)
erki@siil:~$

@maxlath
Copy link

maxlath commented Mar 11, 2014

quite the same problem here

`--> mpsyt                                 
Traceback (most recent call last):
  File "/usr/local/bin/mpsyt", line 297, in <module>
    class Config(object):
  File "/usr/local/bin/mpsyt", line 308, in Config
    DDIR = get_default_ddir()
  File "/usr/local/bin/mpsyt", line 133, in get_default_ddir
    defn = [x for x in lines if x.startswith("XDG_DOWNLOAD_DIR")]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 28: ordinal not in range(128)

@thomasleveil
Copy link
Contributor

@np1 325edff may be the source of this issue.

I also had another similar crash this morning while playing a playlist:
Problem fetching this item: 'ascii' codec can't encode character u'\u2022' in position 96: ordinal not in range(128)

@np1
Copy link
Contributor

np1 commented Mar 11, 2014

I can reproduce the original error reported in this issue by changing my download folder set in ~/.config/user-dirs/dirs to `XDG_DOWNLOAD_DIR="$HOME/DownloadsÉ"``)

Reverting commit https://github.com/np1/mps-youtube/commit/325edff68 fixes this, although it reintroduces the problem we had with Python 3 in the code for the status line:

`File "./mpsyt", line 1300, in mplayer_status
    if char in '\r\n':
TypeError: 'in <string>' requires string as left operand, not bytes

I will try to find a solution.

@np1 np1 added the bug label Mar 11, 2014
@np1
Copy link
Contributor

np1 commented Mar 11, 2014

Hi, for those experiencing this issue, please try the version on this branch: https://github.com/np1/mps-youtube/tree/issue50

You can download just the modified mpsyt here: https://raw.github.com/np1/mps-youtube/issue50/mpsyt

and run it with python mpsyt

Please let me know how it goes. Thanks.

@erkiha
Copy link
Author

erkiha commented Mar 11, 2014

This is working fine, thanx.

@np1
Copy link
Contributor

np1 commented Mar 11, 2014

Good, thanks for testing. I will release a bug-fix version on PyPI and Github master branch soon if no other issues arise.

@maxlath
Copy link

maxlath commented Mar 11, 2014

Works fine here too, thanks!

@np1 np1 added the resolved label Mar 11, 2014
@np1
Copy link
Contributor

np1 commented Mar 11, 2014

Thanks, I have released this now (v0.01.40)

@np1 np1 closed this as completed Mar 11, 2014
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

4 participants