-
Notifications
You must be signed in to change notification settings - Fork 86
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
Python3 support #614
Comments
Hi! thanks for taking the time to reach out. Apologies for such a slow reply. I've been really busy in the last few months, I'm now slowly starting to catch up. The migration to Python3 is something that is high in my list. I'm planning to start looking into this soon. I hoping that at this time, all of Tizonia's Python dependencies are now Python3 friendly, so I suppose there are no impediments to migrate Tizonia as well. So I'm marking this as enhancement, but hopefully to be started soon. |
And of course, if anybody wants to contribute, any help would be greatly appreciated! |
This is in progress in the python3 branch |
|
Thanks for bringing this to my attention. It is now fixed in the develop branch. The Python3 branch was merged into 'develop' a few days ago, hopefully I will be cutting a new release of Tizonia soon. |
[Google Play Music] (TypeError) : write() argument must be str, not bytes |
which command reproduces it? Is that an error in 'develop'? You can export this to see the exception line number:
|
[Google Play Music] (TypeError) : write() argument must be str, not bytes
Traceback (most recent call last):
File "/usr/lib/python3.8/site-packages/tizgmusicproxy.py", line 191, in
__init__
pickle.dump(self.__gmusic.session._authtoken, f)
TypeError: write() argument must be str, not bytes
…On Thu, Nov 28, 2019 at 3:35 PM Juan A. Rubio ***@***.***> wrote:
which command reproduces it? Is that an error in 'develop'?
You can export this to see the exception line number:
export TIZONIA_GMUSICPROXY_DEBUG=1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#614?email_source=notifications&email_token=ABIPVCCWPLNBDGB2WCRBJMDQWATRPA5CNFSM4ILLTZ22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFNM2XQ#issuecomment-559598942>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABIPVCFTBJO7NJJF33PQGBDQWATRPANCNFSM4ILLTZ2Q>
.
|
Thanks!, I think this is fixed by 23ce594 |
I don't know what this is categorized, but, I got a coredump
|
Would it be possible to have a full stack trace? |
I think the thing is occurring in "mpris"
|
|
Yeah, it seeems to be mpris and dbus |
uhg.. yes, the DBUS MPRIS implementation is a bit buggy. Does that go away if you disable it in your config file?
|
yes
…On Fri, Nov 29, 2019 at 11:24 AM Tizonia ***@***.***> wrote:
uhg.. yes, the DBUS MPRIS implementation is a bit buggy.
Does that go away if you disable it in your config file?
mpris-enabled = true
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#614?email_source=notifications&email_token=ABIPVCEBLZZXYJRRXUJN3ZDQWE62VA5CNFSM4ILLTZ22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFPHD7I#issuecomment-559837693>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABIPVCD6MHKOXCWZAR3L7PDQWE62VANCNFSM4ILLTZ2Q>
.
|
https://github.com/jamesturk/validictory can you move the code that use validictory to jsonschema? |
If I'm not mistaken, validictory is an indirect dependency. Please see this pull request in the gmusicapi project: |
Closing this issue for now. The code is now in the master branch, to be released very soon. |
do you know when you'll be able to port the python2 parts of the app to python3
The text was updated successfully, but these errors were encountered: