Do not try to start pulseaudio (and throw error) if it is missing.
Maybe create pulseaudio = auto
mode and use it by default?
pulseaudio is installed in 99.9% of desktop distributions, it is an install dependency (or recommended at least), and it is required to get sound forwarding working properly on the server side.
If your setup is non-standard, the error message tells you what option to change.
I think that's more than enough for now. I may still close this as wontfix.
pulseaudio is installed in 99.9% of desktop distributions.
That is a bold assumption. I've just quickly checked debian DE metapackages for LXDE, LXQt, Mate and Cinnamon, none of them tries to install pulseaudio by dependency. And luckily, xpra package doesn't have such dependancy too.
Anything but screen and clipboard forwarding is a bonus feature anyway :)
Done in r15118 + minor fix in r15119. One benefit of this change is that we no longer need "shell=True" to execute pulseaudio.
@psycho_zs: please close if that works for you.
Thanks, I'll test it with the next beta build.
No more errors with default 'pulseaudio' setting and no pulseaudio. BTW, you would probably want to add python-gst-1.0 to Recommends of the package.
FYI: python-gst-1.0
is already in the "Recommends" list.
this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/1209