[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-19 Thread Rodrigo Moya
It is Bad State indeed, I guess this happens during the time it tries to reconnect to pulseaudio, so I would call this fixed with my latest additions to the branch, as we would need to remove the streams/sinks/etc outside of the idle_reconnect, and I'm not sure what the effect would be. At least

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-19 Thread David Henningsson
Here's what I think: Neither us can reproduce the crash. Put your fix in a ppa so affected people can test and see if that helps against the crashes. If it is helpful, we can SRU the fix. -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-19 Thread Rodrigo Moya
Upstream accepted the patch, so I'm proposing my branch for merging/upload. I think it's safe, since I could replicate the crash once (only once though) without this patch and been unable to with this branch -- You received this bug notification because you are a member of Ubuntu Desktop Bugs,

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-19 Thread Launchpad Bug Tracker
This bug was fixed in the package gnome-settings-daemon - 2.32.1-0ubuntu13.1 --- gnome-settings-daemon (2.32.1-0ubuntu13.1) natty; urgency=low * debian/patches/17_react_when_streams_removed.patch: - Add patch to react when mixer streams are removed (LP: #750334) -- Rodrigo

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-18 Thread David Henningsson
Built the package, installed it and rebooted. After reboot, media keys stops working immediately after boot (perhaps when the login sound stops playing?) But perhaps your fix is on the right track nonetheless, and that you could just add a check against what stream is being removed (i e if id's

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-18 Thread Rodrigo Moya
Right, should be indeed checking the id (I saw it on Friday, but had inet connection problems so couldn't push), so please pull the branch again, it should be working ok now (at least for me it seems to) -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-18 Thread David Henningsson
Result of testing your latest patch version - if I press media keys within ~5 seconds (which probably corresponds to the 5 second timer to idle_reconnect), I still get the warning message (and OSD), after that, I get neither warning message nor OSD. After bringing PA up again things works

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-18 Thread Rodrigo Moya
The same warning as before? What does that warning mean (in English)? -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-settings-daemon in Ubuntu. https://bugs.launchpad.net/bugs/750334 Title: gnome-settings-daemon crashed with

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-18 Thread David Henningsson
Yes, and the warning translates to Invalid state. In this case it could just as well mean assertion failure, I guess. -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-settings-daemon in Ubuntu.

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-15 Thread David Henningsson
The dereference of a dangling pa_context should be fixed in gnome- settings-daemon rather than PA. ** Package changed: pulseaudio (Ubuntu) = gnome-settings-daemon (Ubuntu) -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-15 Thread Rodrigo Moya
** Changed in: gnome-settings-daemon (Ubuntu) Status: Triaged = In Progress ** Changed in: gnome-settings-daemon (Ubuntu) Assignee: (unassigned) = Rodrigo Moya (rodrigo-moya) -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-15 Thread Launchpad Bug Tracker
** Branch linked: lp:~rodrigo-moya/ubuntu/natty/gnome-settings- daemon/fix-750334 -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-settings-daemon in Ubuntu. https://bugs.launchpad.net/bugs/750334 Title: gnome-settings-daemon

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-15 Thread Rodrigo Moya
JeSTeR7: can you easily replicate this problem? If so, could you please build and test the attached branch to see if it fixes it for you? That is: $ bzr get lp:~rodrigo-moya/ubuntu/natty/gnome-settings-daemon/fix-750334 $ sudo apt-get install bzr-builddeb $ cd fix-750334 $ bzr bd -- -b this will

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-15 Thread David Henningsson
Hey Rodrigo and thanks for looking at this bug. A quick review (haven't tested yet) - isn't this patch a little much, e g if you plug in usb headphones, and then unplug them, wouldn't that cause the media keys to stop working? -- You received this bug notification because you are a member of

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-15 Thread Rodrigo Moya
David, it shouldn't, since it will re-create the default sink when the headphones are unplugged. But good point anyway, so yes, will run with it over the weekend to make sure nothing breaks. Can you please do the same for a couple of days or so? -- You received this bug notification because you

[Bug 750334] Re: gnome-settings-daemon crashed with SIGABRT in raise()

2011-04-08 Thread Sebastien Bacher
** Visibility changed to: Public ** Package changed: gnome-settings-daemon (Ubuntu) = pulseaudio (Ubuntu) -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-settings-daemon in Ubuntu. https://bugs.launchpad.net/bugs/750334 Title: