-
Notifications
You must be signed in to change notification settings - Fork 31
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
Linux mentioned, but not possible to get working? #108
Comments
The win32 module is only used for supporting long and network hosted files on Windows, and the import is skipped on other platforms. And yes on Linux it's the .so version of the library not the .dll, support for either us handled by pythonmpv. Looks like removing the win32 line from the requirements.txt should get it all installing for you. I'll see about adding in a Linux specific requirements.txt |
OK, I had further problems when I tried that originally, but didn't realise most of the python stack dumps were from code paths that were try/excepted. Still unable to test probably because of my libmpv version (libmpv1) - I can't test that until I'm in a position to update my workstation from debian oldstable. |
Is it still not possible to run this on Linux? I recently put Manjaro and I'd love to have this on it. I really like the tool. |
@LeKite I've added a requirements-linux.txt now skipping the win32 dependency, that along with libmpv2 should be all you need, give it a go and if you get any failures we'll see what additional guidance we need to add to a linux install note. |
Mmm im very stupid to know how to make it run though. An AUR package would be awesome |
There's several mentions on Linux/ubuntu etc in eg README.md, but judging from requirements.txt, this is never going to work with the dependency on pywin32 and inherent dependencies on things such as mpv-2.dll. Looking at
git log
/git blame
, it looks like linux has been an afterthought since well before 4420ee3 and maybe all the way back to v2.1.4 (which I couldn't getpip3 install -r requirements.txt
to run anyway).If it's no longer a consideration, maybe that should be clarified in README.md.
The text was updated successfully, but these errors were encountered: