xpra icon
Bug tracker and wiki

Opened 13 months ago

Closed 4 months ago

Last modified 4 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 13 months 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 13 months 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 4 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 4 months ago by Antoine Martin

Owner: changed from Antoine Martin to psycho_zs

comment:5 Changed 4 months ago by psycho_zs

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

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

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

Note: See TracTickets for help on using tickets.