padsp should only route sound through pulseaudio if it is running
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
pulseaudio (Ubuntu) |
Fix Released
|
Wishlist
|
Unassigned |
Bug Description
Binary package hint: pulseaudio
currently the padsp program has:
if [ x"$LD_PRELOAD" = x ] ; then
LD_PRELOAD=
else
LD_PRELOAD=
fi
export LD_PRELOAD
exec "$@"
it should really do:
if ps ax | grep -v grep | grep pulseaudio > /dev/null; then
if [ x"$LD_PRELOAD" = x ] ; then
else
fi
fi
export LD_PRELOAD
exec "$@"
or with whatever test you want to use to see if pulse is running. This means applications can then actually have padsp in the exec line of .desktop files, and will work with pulse running, and without. the variations of ubuntu would all have to make sure that the padsp file is always available (ie always install pulseaudio-utils)
Alternatively, you could make another script not part of the pulseaudio package which optionally runs padsp if and only if pulseaudio is running. ie a script called padsp1, run as "padsp1 application" and containing:
if ps ax | grep -v grep | grep pulseaudio > /dev/null; then
exec "padsp $@"
else
exec "$@"
fi
Related branches
- Luke Yelavich: Pending requested
Changed in pulseaudio: | |
importance: | Undecided → Wishlist |
status: | New → Incomplete |
Changed in pulseaudio: | |
status: | Incomplete → Fix Committed |
This bug was fixed in the package pulseaudio - 0.9.14-0ubuntu2
---------------
pulseaudio (0.9.14-0ubuntu2) jaunty; urgency=low
* Add fixes from trunk: driver_ error_redirect_ to_alsa_ devs.patch padsp_wrapper_ check.patch (LP: #269939) pulseaudio. manpages: pulseaudio- esound- compat. manpages: pulseaudio- utils.links:
- 0012_clarify_
* Add 0013_add_
* debian/
debian/
debian/
- Install upstream man pages (LP: #219669, #318660),
- Link to pacat for parec man page (LP: #294628)
-- Daniel T Chen <email address hidden> Thu, 22 Jan 2009 02:57:54 -0500