xpra icon
Bug tracker and wiki

Opened 2 years ago

Closed 14 months ago

Last modified 14 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:

Description

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

comment:1 Changed 2 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 2 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 14 months 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 14 months ago by Antoine Martin

Owner: changed from Antoine Martin to psycho_zs

comment:5 Changed 14 months ago by psycho_zs

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

comment:6 Changed 14 months 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 14 months ago by Antoine Martin

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

Note: See TracTickets for help on using tickets.