Package Details: syncplay 1.6.1-1

Git Clone URL: https://aur.archlinux.org/syncplay.git (read-only)
Package Base: syncplay
Description: synchronize watching movies on mplayer2, vlc, mpv, and mpc-hc on many computers
Upstream URL: http://syncplay.pl/
Licenses: custom
Submitter: blaenk
Maintainer: blaenk (alexandria)
Last Packager: alexandria
Votes: 15
Popularity: 0.066102
First Submitted: 2014-02-22 05:52
Last Updated: 2018-12-22 22:32

Pinned Comments

alexandria commented on 2018-12-22 22:34

NOTICE: I have updated this package to list pyside2 and qt5-declarative as OPTIONAL. If you are using Syncplay for GUI functions (which you most likely are) -- please install these by hand with sudo pacman -S pyside2 qt5-declarative, or an equivalent method for your setup.

Latest Comments

1 2 3 Next › Last »

alexandria commented on 2018-12-22 22:34

NOTICE: I have updated this package to list pyside2 and qt5-declarative as OPTIONAL. If you are using Syncplay for GUI functions (which you most likely are) -- please install these by hand with sudo pacman -S pyside2 qt5-declarative, or an equivalent method for your setup.

blaenk commented on 2018-11-19 00:46

Sounds good :)

alexandria commented on 2018-11-18 23:27

Okies! Apparently I can pin comments in this thread too as a co-maintainer, so I'll just do that whenever I commit the changes. ^.^

blaenk commented on 2018-11-18 21:55

There's probably no need to be honest, if it's the last thing normally printed when installing the package. Otherwise if there are other things printed afterward then it could be easy to lose in the fold, especially maybe with certain AUR helpers when installing or updating many other packages, but I guess we have no control over that. But yeah there's probably no need. I'll leave it to your discretion.

My main concern is that existing users might be caught by surprise with the change, and new users alike. They might install the package and expect to see a "Syncplay" icon somewhere in their list of applications (or they might have file types associated with Syncplay already) but not see it anywhere.

So yeah hopefully they would read the post-install messages. But it's not a big deal. Worst case they might come to this page and ask about it, which gives me the idea, if you like you can also post a concise message here announcing the new change and information on how to the GUI working, and I can pin it to the top of the messages.

I think that should be good enough on our part.

alexandria commented on 2018-11-18 21:42

*******************************************************************

* How should we make the post-install message prominent? Is there a guideline on

* that? If not, I'll probably just have the terminal print out lots of stars,

* like this.

*******************************************************************

alexandria commented on 2018-11-18 21:34

It also appears that .pyc optimized bytecode files aren't being generated by the Syncplay team's install toolchain, which is unfortunate. I'll shoot the Syncplay team a message about that and see what their equivalent of ./setup.py --optimize=1 is.

alexandria commented on 2018-11-18 21:25

Fair enough!

blaenk commented on 2018-11-18 21:23

@alexandria I think there are more clients than servers, and it's more likely that clients will want and expect the GUI.

That said, I'm fine with this change as long as the post-install message is prominent and very clear as to how to get the GUI working. If enough people express confusion or frustration then I will want to revert this change, but in the meantime, for sure go ahead :)

alexandria commented on 2018-11-18 21:18

@blaenk I hope you don't mind, but I really do think we should update this package to not require pyside2. Not only does the server feature not require it -- it appears that a headless mode for the syncplay client has also been added, which appears to work quite well. Thus, I am rewriting the MAKEPKG to have pyside2 and qt5-declarative as optdepends instead of depends, and to simply warn the user during installation that they need to install these packages if they want the GUI.

Please let me know if you don't think I should commit these changes!

alexandria commented on 2018-11-18 06:02

Also, you're welcome. :D