https://bugzilla.redhat.com/show_bug.cgi?id=1225229 Christoph Wickert <cwickert@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cwickert@xxxxxxxxxxxxxxxxx --- Comment #11 from Christoph Wickert <cwickert@xxxxxxxxxxxxxxxxx> --- 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. 2. You will also need this patch: https://github.com/the-cavalry/light-locker/commit/1c23309c2f6a 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. -- 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