xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.

Opened 6 years ago

Closed 5 years ago

Last modified 12 months ago

#1209 closed defect (fixed)

pulseaudio presence dedection

Reported by: psycho_zs Owned by: psycho_zs
Priority: minor Milestone: 2.0
Component: sound Version: trunk
Keywords: Cc:


Do not try to start pulseaudio (and throw error) if it is missing.

Maybe create pulseaudio = auto mode and use it by default?

Change History (8)

comment:1 Changed 6 years ago by Antoine Martin

Component: androidsound
Milestone: 0.18future
Priority: majorminor

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.

comment:2 Changed 6 years ago by psycho_zs

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 :)

comment:3 Changed 5 years ago by Antoine Martin

Milestone: future2.0

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.

comment:4 Changed 5 years ago by Antoine Martin

Owner: changed from Antoine Martin to psycho_zs

comment:5 Changed 5 years ago by psycho_zs

Thanks, I'll test it with the next beta build.

comment:6 Changed 5 years ago by psycho_zs

Resolution: fixed
Status: newclosed

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.

comment:7 Changed 5 years ago by Antoine Martin

FYI: python-gst-1.0 is already in the "Recommends" list.

comment:8 Changed 12 months ago by migration script

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/1209

Note: See TracTickets for help on using tickets.