padsp should only route sound through pulseaudio if it is running

[Duncan Hawthorne]

Binary package hint: pulseaudio

currently the padsp program has:

if [ x"$LD_PRELOAD" = x ] ; then
   LD_PRELOAD="libpulsedsp.so"
else
   LD_PRELOAD="$LD_PRELOAD libpulsedsp.so"
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
       LD_PRELOAD="libpulsedsp.so"
    else
       LD_PRELOAD="$LD_PRELOAD libpulsedsp.so"
    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

[Launchpad Janitor (janitor)]

This bug was fixed in the package pulseaudio - 0.9.14-0ubuntu2

---------------
pulseaudio (0.9.14-0ubuntu2) jaunty; urgency=low

  * Add fixes from trunk:
    - 0012_clarify_driver_error_redirect_to_alsa_devs.patch
  * Add 0013_add_padsp_wrapper_check.patch (LP: #269939)
  * debian/pulseaudio.manpages:
    debian/pulseaudio-esound-compat.manpages:
    debian/pulseaudio-utils.links:
    - 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