Closed edanluna closed 8 years ago
I don't use OS X. If /Applications/VLC.app/Contents/MacOS/VLC is the wrong path then what is the 'complete path of VLC' that you reckon Syncplay should be suggesting?
My bad, that is not the issue. The issue is that Syncplay chooses /Applications/VLC.app/
(at least in my computer) when I select VLC in the Open Dialog as the full path for VLC, instead of /Applications/VLC.app/Contents/MacOS/VLC
.
When I click in the arrow right to the textbox THEN I see the full path as an option.
Steps to reproduce:
/Applications/VLC.app
as contentExpected behavior: Either:
/Applications/VLC.app
in the textbox deduce the rest of the path and either:
All in all is just an UI/Usability issue.
As a python newbie what happened to me is that I saw the error in the terminal first, then I saw that mpv didn't have that problem at all. So went and compared the code of both classes in the Syncplay code until I figured out that both players VLC and mpv have the same code for expanding the path so it couldn't be something that the player Factory or anything like that could be doing (I avoid UI code as much as I can) and decided to make this ticket and THEN I saw the drop down arrow where the full path of VLC was instead of the short one that have been plaguing me.
Thanks for bringing this issue to light. I've mentioned this problem at https://github.com/Syncplay/syncplay/issues/81 so hopefully we'll find someone who will be able to code and test a fix for this. For people who use a Mac and know Python I'm guessing it won't be too hard to solve based on your suggested expected behaviour. If it is possible, just correcting the location as appropriate would be nicer than having to give a warning.
Hopefully nilsding's commit fixes this problem :)
I follow the instructions to install syncplay. With mpv I can see the movie but with VLC y get:
After some exploration syncplay tries to run
/Application/VLC.app
which is the text that is show by default in the UI. I assume it is related.I don't know the reason for this to happen or if you don't consider a problem but when I go back and select the complete path of VLC it works. I'm not used to how mac works but I would assume that some people will find the error and it is just an UI issue, to make it clear that you have to select the full path of the binary or to deduce the path of the binary.