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

[OS X] Can't add files to shared playlist. #146

Closed
Duamer opened this Issue Sep 15, 2017 · 8 comments

Comments

Projects
None yet
3 participants
@Duamer

Duamer commented Sep 15, 2017

This problem occurred in two different systems I have. A clean slate system that only had Syncplay installed and my main, lived-in system.

I can'd add files to the shared playlist. Neither drag-and-drop, nor right-click, "Add file(s) to bottom of playlist" function. I momentarily see a files browser window pop up when I right-click.
I can add URLs to the playlist and they work as they should. I've setup the media directories and had a different user make the playlist and that works as well.

@albertosottile

This comment has been minimized.

Show comment
Hide comment
@albertosottile

albertosottile Sep 15, 2017

Member

Confirmed. Looking for a workaround. Thanks.

Member

albertosottile commented Sep 15, 2017

Confirmed. Looking for a workaround. Thanks.

albertosottile added a commit to albertosottile/syncplay that referenced this issue Sep 15, 2017

@albertosottile

This comment has been minimized.

Show comment
Hide comment
@albertosottile

albertosottile Sep 15, 2017

Member

If you have time, please test the workaround that I have just pushed to my branch: https://github.com/alby128/syncplay . Right now, I tested it on 10.12.6 and it works for me. EDIT: please keep in mind that it requires to install pyobjc (e.g. pip install pyobjc).

Member

albertosottile commented Sep 15, 2017

If you have time, please test the workaround that I have just pushed to my branch: https://github.com/alby128/syncplay . Right now, I tested it on 10.12.6 and it works for me. EDIT: please keep in mind that it requires to install pyobjc (e.g. pip install pyobjc).

@Duamer

This comment has been minimized.

Show comment
Hide comment
@Duamer

Duamer Sep 15, 2017

How do I install pyobjc with homebrew? I used the homebrew method of your directions.

Incase that's not what's causing my error, here's what I got:

users-iMac:syncplay user$ python2 syncplayClient.py
2017-09-15 19:22:39.429 Python[628:5526] modalSession has been exited prematurely - check for a reentrant call to endModalSession:
Traceback (most recent call last):
File "syncplayClient.py", line 19, in
SyncplayClientManager().run()
File "/Users/user/syncplay/syncplay/clientManager.py", line 9, in run
interface = ui.getUi(graphical=not config["noGui"])
File "/Users/user/syncplay/syncplay/ui/init.py", line 9, in getUi
ui = GraphicalUI()
NameError: global name 'GraphicalUI' is not defined
Segmentation fault: 11

Duamer commented Sep 15, 2017

How do I install pyobjc with homebrew? I used the homebrew method of your directions.

Incase that's not what's causing my error, here's what I got:

users-iMac:syncplay user$ python2 syncplayClient.py
2017-09-15 19:22:39.429 Python[628:5526] modalSession has been exited prematurely - check for a reentrant call to endModalSession:
Traceback (most recent call last):
File "syncplayClient.py", line 19, in
SyncplayClientManager().run()
File "/Users/user/syncplay/syncplay/clientManager.py", line 9, in run
interface = ui.getUi(graphical=not config["noGui"])
File "/Users/user/syncplay/syncplay/ui/init.py", line 9, in getUi
ui = GraphicalUI()
NameError: global name 'GraphicalUI' is not defined
Segmentation fault: 11

@albertosottile

This comment has been minimized.

Show comment
Hide comment
@albertosottile

albertosottile Sep 15, 2017

Member

To install pyobjc on your homebrew, just run pip2 install pyobjc. However, I have to admit that your error message puzzles me a little... How did you get it? When you tried to add something to a shared playlist?

Member

albertosottile commented Sep 15, 2017

To install pyobjc on your homebrew, just run pip2 install pyobjc. However, I have to admit that your error message puzzles me a little... How did you get it? When you tried to add something to a shared playlist?

@Duamer

This comment has been minimized.

Show comment
Hide comment
@Duamer

Duamer Sep 16, 2017

Looks like that fixed it. I'll be able to test it by Monday to see if it works with other users in the room.

I got that error when I clicked "Store configuration and run Syncplay."

Forgot to mention, I'm running 10.11.6.

Duamer commented Sep 16, 2017

Looks like that fixed it. I'll be able to test it by Monday to see if it works with other users in the room.

I got that error when I clicked "Store configuration and run Syncplay."

Forgot to mention, I'm running 10.11.6.

@Et0h Et0h added bug macOS labels Sep 16, 2017

albertosottile added a commit to albertosottile/syncplay that referenced this issue Sep 17, 2017

@albertosottile

This comment has been minimized.

Show comment
Hide comment
@albertosottile

albertosottile Sep 18, 2017

Member

@Duamer Did you test this fix with other users?

Member

albertosottile commented Sep 18, 2017

@Duamer Did you test this fix with other users?

@Duamer

This comment has been minimized.

Show comment
Hide comment
@Duamer

Duamer Sep 18, 2017

Testing now and it looks good. I was able to setup the room and the order user joined. Works like it does when I was using the windows client.

Duamer commented Sep 18, 2017

Testing now and it looks good. I was able to setup the room and the order user joined. Works like it does when I was using the windows client.

@Duamer Duamer closed this Sep 18, 2017

@albertosottile

This comment has been minimized.

Show comment
Hide comment
@albertosottile

albertosottile Sep 19, 2017

Member

@Duamer Thanks, but please reopen this, as it has only been fixed in my fork. To fix it here, we still need to merge the PR #148 .

Member

albertosottile commented Sep 19, 2017

@Duamer Thanks, but please reopen this, as it has only been fixed in my fork. To fix it here, we still need to merge the PR #148 .

@Duamer Duamer reopened this Sep 19, 2017

Et0h added a commit that referenced this issue Sep 28, 2017

@Et0h Et0h closed this Oct 1, 2017

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