FS#36290 - [rhythmbox] while playing, disables automatic screen lock

Attached to Project: Arch Linux
Opened by Luis Henrique Mello (lmello) - Friday, 26 July 2013, 20:32 GMT
Last edited by Gerardo Exequiel Pozzi (djgera) - Friday, 18 April 2014, 02:27 GMT
Task Type Bug Report
Category Upstream Bugs
Status Closed
Assigned To Jan de Groot (JGC)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description:
Well I really don't know if this is a bug or a 'feature', but it used to work before. I think some recent upgrades of other packages triggered this issue, because it's the same package version I've used for months.

Additional info:
rhythmbox 2.99.1-2

Steps to reproduce:
Open rhythmbox, set to automatic lock screen for about 1 minute or so, an play a stream or audio file. The desktop does not lock up automatically like it should.
This task depends upon

Closed by  Gerardo Exequiel Pozzi (djgera)
Friday, 18 April 2014, 02:27 GMT
Reason for closing:  Fixed
Additional comments about closing:  upstream
Comment by Luis Henrique Mello (lmello) - Friday, 26 July 2013, 22:38 GMT
Another thing: start playing something on Rhythmbox, manually lock the screen, and click on the 'next' or 'previous' button that appears. Rhythmbox crashes immediatly.

Attached is what gdb spits when I do this.

gobject-introspection (which contains was updated on July 16, so I'll try to downgrade it.

edit: downgrade the package to the previous repo version, gdb spits this instead:

'Program received signal SIGABRT, Aborted.
0x00007ffff6c7a1c9 in raise () from /usr/lib/libc.so.6'
   debug (0.1 KiB)
Comment by Luis Henrique Mello (lmello) - Friday, 26 July 2013, 23:10 GMT
Rebuilding the package solves the more serious bug that causes rhythmbox to crash upon commands in the lock screen, but it still disables auto-locking preferences.
Comment by Luis Henrique Mello (lmello) - Wednesday, 31 July 2013, 14:57 GMT
I DISABLED one plugin which is enabled by default:

- MPRIS D-Bus interface

And then I got my screen locked as usual with rhythmbox playing.

I have dbus 1.6.12-1 installed.

BUT, if I restart Rhythmbox, even with this plugin disabled, the screen does not lock again.
Comment by Mark H (Peaceseeker) - Thursday, 17 October 2013, 13:05 GMT
This behaviour is caused by the 'power-manager plugin'. This plugin, for some reason is supplied as 'built in' with the source, and so the plugin cannot be disabled from the plugin menu, as it is not visible. See this bug report (https://bugs.launchpad.net/ubuntu/+source/rhythmbox/+bug/912912), not the same exactly, but similar (visibility) issue.

You can make the plugin visible easily by opening, as root:
/usr/lib/rhythmbox/plugins/power-manager/power-manager.plugin

and change the line:
Builtin=true
to
Builtin=false

I'm unsure if this plugin being hidden is intentional?, but I am certain most users would like the plugin visible so that they can toggle enabled/disabled as desired, I still consider this a bug, as the solution is virtually effortless, and the plugin is functional.
Comment by Jan de Groot (JGC) - Thursday, 17 October 2013, 13:51 GMT
The description of the plugin doesn't match what it's doing. The plugin is supposed to inhibit suspend, which would be done by passing GTK_APPLICATION_INHIBIT_SUSPEND. What it does however is passing GTK_APPLICATION_INHIBIT_IDLE, which means the session manager will never kick in the screensaver.
Comment by Mark H (Peaceseeker) - Thursday, 17 October 2013, 15:08 GMT
Ah that's why, I recompiled with GTK_APPLICATION_INHIBIT_SUSPEND, and it no longer effects the screen lock. So it seems like it could be a mistake or something overlooked in the code, as already mentioned its action does not match the description.
Comment by Luis Henrique Mello (lmello) - Tuesday, 29 October 2013, 15:38 GMT
@Mark H,

How did you recompile? Could you please provide a patch? I could send it upstream (https://bugzilla.gnome.org/show_bug.cgi?id=705105) then.
Comment by Mark H (Peaceseeker) - Tuesday, 29 October 2013, 18:37 GMT
I recompiled with a couple of SED lines added to the PKGBUILD: http://pastebin.com/szFhNUnf. The differences(patch) are just two lines shown here: http://pastebin.com/3w2nifQZ. I wasn't sure if the hidden nature of the plugin was intended or not. As the fix is so simple I'm guessing the intended behaviour is to hide it, although doing this presents more problems, such as this bug report.

Also I have not fully tested 'GTK_APPLICATION_INHIBIT_SUSPEND', all I know is that it allows the screensaver to work. Personally, I prefer to disable the plugin, its a little buggy.
Comment by Luis Henrique Mello (lmello) - Tuesday, 29 October 2013, 23:28 GMT
@Mark H,

Thank you very much. I took the liberty to propose the patch upstream, let's see what the developers can do.

ps: probably the hidden nature of the plugin was indeed intended, but as it is it's passing the wrong thing to the gnome shell.
Comment by Gerardo Exequiel Pozzi (djgera) - Sunday, 16 February 2014, 15:00 GMT
  • Field changed: Status (Assigned → Waiting on Response)
  • Field changed: Category (Packages: Extra → Upstream Bugs)
status?
Comment by Luis Henrique Mello (lmello) - Monday, 17 February 2014, 08:22 GMT
https://mail.gnome.org/archives/rhythmbox-devel/2013-October/msg00005.html

"It looks like GTK_APPLICATION_INHIBIT_SUSPEND prevents laptops from
suspending when the lid is closed, so it's not the right flag to use
here."

Loading...