[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
Fix Released in Unity Unity 7.2.0. ** Changed in: unity Status: Fix Committed = Fix Released -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
https://launchpad.net/ubuntu/+source/unity/7.1.2+14.04.20140311-0ubuntu1 ** Changed in: unity (Ubuntu) Status: Triaged = Fix Released ** Changed in: unity Status: In Progress = Fix Released -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
https://launchpad.net/ubuntu/+source/gnome-screensaver/3.6.1-0ubuntu10 ** Changed in: gnome-screensaver (Ubuntu) Importance: Undecided = High ** Changed in: gnome-screensaver (Ubuntu) Status: Triaged = Fix Released -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
** Changed in: unity Status: Fix Released = Fix Committed -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
This is the patch that is needed in gnome-screensaver in order to get the proper unity integration. ** Also affects: gnome-screensaver (Ubuntu) Importance: Undecided Status: New ** Changed in: gnome-screensaver (Ubuntu) Status: New = In Progress ** Changed in: gnome-screensaver (Ubuntu) Assignee: (unassigned) = Marco Trevisan (Treviño) (3v1n0) ** Patch added: 33_unity_lockscreen_on_lock.patch https://bugs.launchpad.net/ubuntu/+source/gnome-screensaver/+bug/1282798/+attachment/4017030/+files/33_unity_lockscreen_on_lock.patch -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
** Branch linked: lp:~3v1n0/gnome-screensaver/use-unity-lockscreen -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
OK. I've had a look at this request. Thanks for the patch to g-s. I had a brief look at it and I'm not sure why you introduced the lock-request signal? What happens if the lock fails in manager_lock_request? I think you leak the GSignals too. That's only a review from quickly looking at the diff - someone else should review it properly. Anyway, I want to give a chance for this to get into Trusty, but I want to be conservative. So I'll ack this FFe if you can get it in by Wednesday this week (2014-03-12). We're a month away from final freeze now and I'm keen to give this as good a run of user testing as possible before the LTS release. If not, then I don't want to nack the request—find another RT member (I'll be away this week) to re-review the state and see if you can try again. Please keep an eye on bug reports and feedback and if this doesn't go as smoothly as we need then we should be prepared to go back to the known current state and land this solidly next cycle. ** Changed in: unity (Ubuntu) Status: Incomplete = Triaged ** Changed in: gnome-screensaver (Ubuntu) Status: In Progress = Triaged -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
** Patch removed: 33_unity_lockscreen_on_lock.patch https://bugs.launchpad.net/unity/+bug/1282798/+attachment/4017030/+files/33_unity_lockscreen_on_lock.patch -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1282798] Re: [FFe] Provide a lock screen and unlock dialogs in Unity
Thanks for the FFe approval, then: I'm not sure why you introduced the lock-request signal? I didn't want to make GSManager (that is a child of GSMonitor) to keep a reference (even a weak one) of its parent, so the quickest way and the one that wouldn't have involved too many changes to the GS code, was to emit a signal that returns a value if the lock is handled. So, in case unity is running and a lock has been requested, if nothing fails, unity will handle it and GS will ignore the lock. What happens if the lock fails in manager_lock_request? I think you leak the GSignals too. If something fails here, the signal will return FALSE, an thus, the normal GS locking will happen; in case unity will be back (if it has ever been there), then we replace the g-s locking with the native unity locking again. I think you leak the GSignals too. You mean the connections? Btw I've pushed a new version of the patch... ** Patch added: 33_unity_lockscreen_on_lock.patch https://bugs.launchpad.net/unity/+bug/1282798/+attachment/4017110/+files/33_unity_lockscreen_on_lock.patch -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to gnome-screensaver in Ubuntu. https://bugs.launchpad.net/bugs/1282798 Title: [FFe] Provide a lock screen and unlock dialogs in Unity To manage notifications about this bug go to: https://bugs.launchpad.net/unity/+bug/1282798/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs