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

Linux mentioned, but not possible to get working? #108

Open
spacelama opened this issue Nov 10, 2023 · 5 comments
Open

Linux mentioned, but not possible to get working? #108

spacelama opened this issue Nov 10, 2023 · 5 comments

Comments

@spacelama
Copy link

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 get pip3 install -r requirements.txt to run anyway).

If it's no longer a consideration, maybe that should be clarified in README.md.

@dfaker
Copy link
Owner

dfaker commented Nov 10, 2023

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

@spacelama
Copy link
Author

spacelama commented Nov 10, 2023

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.
(install instructions for linux could probably do with a note to ensure apt install libmpv2 or equivalent if available)

@LeKite
Copy link

LeKite commented Jan 16, 2024

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.

@dfaker
Copy link
Owner

dfaker commented Mar 18, 2024

@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.

@LeKite
Copy link

LeKite commented Apr 11, 2024

Mmm im very stupid to know how to make it run though. An AUR package would be awesome

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

3 participants