https://bugzilla.redhat.com/show_bug.cgi?id=1225229 Raphael Groner <projects.rg@xxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |MODIFIED --- Comment #12 from Raphael Groner <projects.rg@xxxxxxxx> --- (In reply to Christoph Wickert from comment #11) > Raphael, your fixes for the desktop file don't seem correct: > > 1. First you fix NotShowIn and then you use sed to remove the key, so your > patch becomes useless. Yeah, this is currently b0rken, as mentioned in the comments. > 2. You will also need this patch: > https://github.com/the-cavalry/light-locker/commit/1c23309c2f6a Thanks. > Please also configure with > > --enable-lock-on-suspend=on > > instead of > > --enable-lock-on-suspend=yes > > because the Red Hat security response team wants machines to be locked after > waking up from suspend (see bug 614608 and bug 632978). "on" means we do it > by default whereas "yes" means just having the option to do it, but that > requires changing the command line in the desktop file that starts > light-locker. Sure. Thanks a lot for these hints. Next update will include both fixes. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review