[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-02-01 Thread Martin Pitt
Thanks, Sebastian: esound (0.2.38-0ubuntu8) hardy; urgency=low * util.c: Fix memory leak in our patch to support multiple users. Thanks to Sebastian Keller for spotting this! (See LP #177072) -- Martin Pitt <[EMAIL PROTECTED]> Fri, 01 Feb 2008 17:57:00 +0100 I also updated http://people

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-21 Thread Sebastian Keller
Yes, I just tested it seems it was the patch that was dropped by mistake. It works now. However I'm still not sure if this patch has a small memory leak, because everytime the function is called, memory for dirname is allocated. This is never freed, because in the original function it would only b

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-21 Thread Daniel T Chen
This is fixed with seb's upload of -0ubuntu7. ** Changed in: esound (Ubuntu) Status: Triaged => Invalid -- AUDIODEV should be exported when using pulseaudio https://bugs.launchpad.net/bugs/177072 You received this bug notification because you are a member of Ubuntu Bugs, which is the bug

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-10 Thread Sebastian Keller
In this patch, isn't the removal of the "dirname == NULL" check causing a small leak, since the space will be allocated everytime the function is called? -- AUDIODEV should be exported when using pulseaudio https://bugs.launchpad.net/bugs/177072 You received this bug notification because you are

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-10 Thread mlind
Was this pitti's patch dropped by a mistake --> http://people.ubuntu.com/patches/esound.multiuser.diff ? Without it esound doesn't find the socket created by pulseaudio-esound-compat (in /tmp/.esd-$UID/). -- AUDIODEV should be exported when using pulseaudio https://bugs.launchpad.net/bugs/177

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-08 Thread Sebastien Bacher
Setting an hardy milestone having applications freezing for a minute is not a nice user experience ** Changed in: esound (Ubuntu) Target: None => ubuntu-8.04 -- AUDIODEV should be exported when using pulseaudio https://bugs.launchpad.net/bugs/177072 You received this bug notification beca

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-07 Thread Sebastian Keller
Patching esd_get_socket_dirname in utils.c without #ifdef would also affect the behaviour of the original esound-daemon (if somebody is not using pulse as replacement) as it uses the name from this function for the creation of the socket, too. I'm not sure if this might lead to problems. The idea

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-07 Thread Daniel T Chen
Patching utils.c seems reasonable to me. Whether there should be a separate "libesd0-pulse" raises issues for 6.06 users, e.g., do we wholescale migrate them to said package? What if they are happily using libesd0-alsa and don't wish to switch? ** Changed in: esound (Ubuntu) Sourcepackagename: p

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-05 Thread Sebastian Keller
I just found out that setting AUDIODEV to "-1000" (according to my user-id) results in problems when using sdl with alsa as sound backend. Zsnes for example has no sound, because alsa fails to open the pcm: ALSA lib pcm.c:2145:(snd_pcm_open_noupdate) Unknown PCM -1000 So defining AUDIODEV sessi

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2008-01-02 Thread Pedro Villavicencio
Thanks for your work, Martin may you take a look to it? thanks you!. ** Changed in: pulseaudio (Ubuntu) Importance: Undecided => Low Assignee: (unassigned) => Martin Pitt (pitti) Status: New => Triaged -- AUDIODEV should be exported when using pulseaudio https://bugs.launchpad.net

[Bug 177072] Re: AUDIODEV should be exported when using pulseaudio

2007-12-17 Thread Sebastian Keller
I created the attached file as /etc/X11/Xsession.d/51pulseaudio-esd (i just chose a number before the 55 of the .gnomerc so it could be easily overwritten by that). This way AUDIODEV is set on every login through gdm for every user. Should work for kdm, too, but i haven't tested it. Only progra