don't animate lock-on-suspend when we don't have systemd

[Timo Kluck]

Hi all,

Here's the patch that *actually* fixes this issue.

I somehow don't receive this list's e-mails in my inbox, so I can't
reply to Tim's e-mail directly. So I'll put my comments here:

Thanks for the advice on the virtual terminal. I wasn't able to fix
this lock with DISPLAY=<displayname> gnome-shell --replace, but I
don't see why.

You're right that this probably should be filed upstream. I will do
so. But in my humble opinion, this should be included in our package
asap too. It's a pretty serious data loss problem, even if the
gnome-shell --replace were to help. In the mean time, upstream can
work on (accepting a) patch that only works around this issue in the
non-systemd case.

Best, Timo

2013/3/10 Timo Kluck <tkluck at infty.nl>:
> Hi ubuntu-gnomists,
>
> I've woken my laptop from suspend a couple of times, finding the
> "screen shield" to be halfway down, grabbing keyboard focus, and no
> way to unlock. I had to kill the X server every time.
>
> My theory for how this came about is that gnome-shell expects to be
> able to inhibit suspend for as long as the animation takes, because it
> uses systemd. The workaround I'm trying is to disable animating the
> lock screen when used for suspending. In many use cases, suspend is
> triggered by closing the laptop lid, so this isn't a very invasive
> change anyway.
>
> It's hard to test these kind of racing conditions because they don't
> happen all the time. This means I'm not 100% sure this actually fixes
> the issue.
>
> I'm attaching the necessary patch. I'll also try to get a patched
> version in my ppa (ppa:tkluck/gnome3).
> Best regards,
> Timo Kluck
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dont-animate-lock-on-suspend.patch
Type: application/octet-stream
Size: 993 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-gnome/attachments/20130311/84fc124d/attachment.obj>