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

Compile pytaglib for Python 2 or 3? - Error Trying To Run 2.1.1 #726

Closed
0pLuS0 opened this issue Oct 11, 2020 · 4 comments
Closed

Compile pytaglib for Python 2 or 3? - Error Trying To Run 2.1.1 #726

0pLuS0 opened this issue Oct 11, 2020 · 4 comments
Labels

Comments

@0pLuS0
Copy link

0pLuS0 commented Oct 11, 2020

Nicotine+ version: 2.11
Operating System/Distribution: Slackware 14.2 x86_64

System apps:
python3-3.7.2
taglib-1.10

Describe the bug

I'm assuming for now that I need to compile pytaglib for python3?

Expected behavior

Have 2.1.1 run when I start it...

Steps to reproduce the bug

When I run nicotine 2.1.1 against pytaglib-1.4.6 built against python3 I get this message at the terminal;

Traceback (most recent call last):
File "/usr/lib64/python3.7/site-packages/pynicotine/config.py", line 88, in init
self.parser.read([self.filename], encoding="utf-8")
File "/usr/lib64/python3.7/configparser.py", line 696, in read
self._read(fp, filename)
File "/usr/lib64/python3.7/configparser.py", line 1014, in _read
for lineno, line in enumerate(fp, start=1):
File "/usr/lib64/python3.7/codecs.py", line 322, in decode
(result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe6 in position 1634: invalid continuation byte

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/usr/lib64/python3.7/site-packages/pynicotine/gtkgui/frame.py", line 3034, in OnActivate
port
File "/usr/lib64/python3.7/site-packages/pynicotine/gtkgui/frame.py", line 126, in init
config
File "/usr/lib64/python3.7/site-packages/pynicotine/pynicotine.py", line 111, in init
self.config = Config(config, data_dir)
File "/usr/lib64/python3.7/site-packages/pynicotine/config.py", line 91, in init
self.parser.read([self.filename], encoding="utf-8")
File "/usr/lib64/python3.7/configparser.py", line 696, in read
self._read(fp, filename)
File "/usr/lib64/python3.7/configparser.py", line 1014, in _read
for lineno, line in enumerate(fp, start=1):
File "/usr/lib64/python3.7/codecs.py", line 322, in decode
(result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe6 in position 1634: invalid continuation byte

P.S. Is nicotine suppose to be built against Python 2 or 3, I forget?

@0pLuS0 0pLuS0 added the bug label Oct 11, 2020
@0pLuS0 0pLuS0 changed the title Compile pytaglib for Python 2 or 3? Compile pytaglib for Python 2 or 3? - Error Trying To Run 2.1.1 Oct 11, 2020
@mathiascode
Copy link
Member

Python 2 is no longer used in Nicotine+, and it seems like pytaglib is working fine for you. Looks like there is a problem loading the config file. Can you install python3-chardet if it's not already installed, and attempt to start Nicotine+ again?

@0pLuS0
Copy link
Author

0pLuS0 commented Oct 13, 2020

Hi,

Ahh it's working now! 👍

The only change I saw listed on the website News section was for pytaglib, I didn't see any mention of python3-chardet.

I went back and checked the website and on Github, I still see no mention of this as a new dependency.

Can someone please add into either the website, or Github, or both a Dependencies section for building, so we'll know what is needed in the future?

THANKS

@mathiascode
Copy link
Member

The python3-chardet dependency is only necessary once for converting an old config file to UTF-8, which I assumed was only needed on Windows. Since virtually every other OS defaults to UTF-8, and we haven't received more reports like this, I've opted to just show an error message about installing chardet in Nicotine+ 2.1.2.

@0pLuS0
Copy link
Author

0pLuS0 commented Oct 16, 2020

But I hope in the future, if things change, there can be a Dependencies listed added somewhere...

THANKS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants