Merge lp:~khadgaray/unity-lens-applications/fix-lockdown into lp:unity-lens-applications

Proposed by Ritesh Khadgaray on 2012-07-03
Status: Rejected
Rejected by: Didier Roche on 2019-02-12
Proposed branch: lp:~khadgaray/unity-lens-applications/fix-lockdown
Merge into: lp:unity-lens-applications
Diff against target: 16 lines (+6/-0)
1 file modified
src/daemon.vala (+6/-0)
To merge this branch: bzr merge lp:~khadgaray/unity-lens-applications/fix-lockdown
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing on 2013-04-13
Didier Roche Needs Fixing on 2012-07-05
Mikkel Kamstrup Erlandsen (community) 2012-07-03 Needs Information on 2012-07-04
Review via email:

Description of the change

Allows an admin to disable command run dialog based on gnome security policies.

To post a comment you must log in.

This adds a hard runtime dep on the org.gnome.desktop.lockdown schema. Does this imply a new dependency for the package as well?

review: Needs Information
Ritesh Khadgaray (khadgaray) wrote :

> This adds a hard runtime dep on the org.gnome.desktop.lockdown schema. Does
> this imply a new dependency for the package as well?

Unity already depends on this via libgnome-desktop-3-4, which needs gsettings-desktop-schemas.

-- ritz

Didier Roche (didrocks) wrote :

I would rather the check happening when the lens is started and then listening to the key changing than after you did a search and ready to activate it.

In that case, we should display a friendly message to the user rather than afterwards :)

Also yeah, as told on IRC, the dependency should be specified directly. And it needs tests ;)

review: Needs Fixing
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:287
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
Executed test runs:

Click here to trigger a rebuild:

review: Needs Fixing (continuous-integration)
Didier Roche (didrocks) wrote :

Didn't get any feedback. Rejecting due to aging MP

Unmerged revisions

287. By Ritesh Khadgaray on 2012-07-03

add lockdown support to disable commandline (LP# 813895)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/daemon.vala'
--- src/daemon.vala 2012-04-23 13:58:42 +0000
+++ src/daemon.vala 2012-07-03 16:28:26 +0000
@@ -997,6 +997,12 @@
997 }997 }
998 else998 else
999 {999 {
1000 Settings lockdown = new Settings ("org.gnome.desktop.lockdown");
1001 if (uri.has_prefix ("unity-runner://") && lockdown.get_boolean ("disable-command-line")) {
1002 warning ("disabled by administrator");
1003 return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
1004 }
1000 try {1006 try {
1001 unowned string home_dir = GLib.Environment.get_home_dir ();1007 unowned string home_dir = GLib.Environment.get_home_dir ();
1002 Process.spawn_async (home_dir, args, null, SpawnFlags.SEARCH_PATH, null, null);1008 Process.spawn_async (home_dir, args, null, SpawnFlags.SEARCH_PATH, null, null);


People subscribed via source and target branches