Launchpad has imported 8 comments from the remote bug at
http://bugzilla.libsdl.org/show_bug.cgi?id=1011.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2010-06-25T19:20:31+00:00 Daniel Ellis wrote:

Created attachment 512
Patch to not specify an application name when creating a pulse audio context.

I created the following bug on launchpad and thought I should raise it
upstream.
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859

---------------------------
SDL based applications sometimes display the wrong application name in the 
Sound Preferences dialog when using pulseaudio.

I can see from the code that the SDL pulse module is initiating a new
pulse audio context and passing an application name using the function
get_progname().

The get_progname() function returns the name of the current process.
However, the process name is often not a suitable name to use. For
example, the OpenShot video editor is a python application, and so
"python" is displayed in the Sound Preferences window (see Bug #596504),
when it should be displaying "OpenShot".

PulseAudio allows applications to specify the application name, either
at the time the context is created (as SDL does currently), or by
special environment variables (see
http://www.pulseaudio.org/wiki/ApplicationProperties). If no name is
specified, then pulseaudio will determine the name based on the process.

If you specify the application name when initiating the pulseaudio
context, then that will override any application name specified using an
environment variable.

As libsdl is a library, I believe the solution is for libsdl to not
specify any application name when initiating a pulseaudio context, which
will enable applications to specify the application name using
environment variables. In the case that the applications do not specify
anything, pulseaudio will fall back to using the process name anyway.

The attached patch removes the get_progname() function and passes NULL
as the application name when creating the pulseaudio context, which
fixes the issue.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859/comments/2

------------------------------------------------------------------------
On 2011-01-15T14:42:12+00:00 Moimael-3 wrote:

Any progress on this bug ? It would be nice if we could fix this bug in
openshot !

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859/comments/3

------------------------------------------------------------------------
On 2011-01-24T02:51:58+00:00 Sam Lantinga wrote:

Thanks, your change is in the repository!

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859/comments/4

------------------------------------------------------------------------
On 2011-01-24T02:54:16+00:00 Sam Lantinga wrote:

This is fixed in SDL 1.3.  The code for SDL 1.2 looks very different and
might need additional review.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859/comments/5

------------------------------------------------------------------------
On 2011-02-16T09:13:50+00:00 Sam Lantinga wrote:

This appears to have caused bug 1119.
Can someone take a look and see what's going on here?

Thanks!

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859/comments/6

------------------------------------------------------------------------
On 2011-02-16T10:15:43+00:00 Daniel Ellis wrote:

I suspect allowing NULL as a parameter to pa_context_new was introduced
in a newer version at some point.

So what we need to find out in order to handle old and new versions of
PulseAudio is:-

1. Can we determine the version of PulseAudio at runtime?

2. Which version of PulseAudio introduced the ability to pass NULL as
the application name?


>From the looks of SDL_pulseaudio.c it is already checking the PulseAudio 
>version using:

  #if (PA_API_VERSION < 12)


The doxygen for PulseAudio does not help much:-

http://0pointer.de/lennart/projects/pulseaudio/doxygen/context_8h.html#a2784c754947a97f02c78b73d7b1c2d5f

Does anyone know? Or do we have to go through the PulseAudio source
history?

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859/comments/7

------------------------------------------------------------------------
On 2011-02-24T23:16:40+00:00 Sam Lantinga wrote:

Assigning to Ryan for his audio cleanup.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859/comments/8

------------------------------------------------------------------------
On 2013-05-21T05:16:17+00:00 Sam Lantinga wrote:

Ryan, can you take a look and see if this is still active for the 2.0
release?

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libsdl1.2/+bug/597859/comments/9


** Changed in: libsdl
   Importance: Medium => Critical

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/597859

Title:
  Wrong application name appears in Sound Preferences when using
  pulseaudio

To manage notifications about this bug go to:
https://bugs.launchpad.net/libsdl/+bug/597859/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to