Merge lp:~nick-dedekind/unity8/trusted-socket.prompt-file into lp:unity8

Proposed by Nick Dedekind
Status: Merged
Approved by: Michał Sawicz
Approved revision: 1048
Merged at revision: 1055
Proposed branch: lp:~nick-dedekind/unity8/trusted-socket.prompt-file
Merge into: lp:unity8
Diff against target: 18 lines (+3/-0)
1 file modified
data/unity8.conf (+3/-0)
To merge this branch: bzr merge lp:~nick-dedekind/unity8/trusted-socket.prompt-file
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Michał Sawicz Approve
Thomas Voß (community) Needs Fixing
Review via email: mp+227051@code.launchpad.net

Commit message

Added environment variable to upstart conf for mir trusted socket

Description of the change

 * Are there any related MPs required for this MP to build/function as expected? Please list.
No

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Yes

 * Did you make sure that your branch does not contain spurious tags?
Yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

 * If you changed the UI, has there been a design review?
N/A

To post a comment you must log in.
Revision history for this message
Thomas Voß (thomas-voss) wrote :

MIR_SERVER_PROMPT_FILE does not need the path to be assigned, but just set in the environment. For that, I would propose that we set it to ENABLED to emphasize its semantics.

review: Needs Fixing
Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

> MIR_SERVER_PROMPT_FILE does not need the path to be assigned, but just set in
> the environment. For that, I would propose that we set it to ENABLED to
> emphasize its semantics.

Ah. I thought it was same as MIR_SOCKET.
Fixed

1047. By Nick Dedekind

dont set path. unset in post

1048. By Nick Dedekind

ENABLED->1

Revision history for this message
Michał Sawicz (saviq) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Yup.

 * Did CI run pass? If not, please explain why.
Not yet, but no change to anything affecting.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/unity8.conf'
2--- data/unity8.conf 2014-05-01 14:25:18 +0000
3+++ data/unity8.conf 2014-07-16 15:52:12 +0000
4@@ -44,6 +44,8 @@
5 rm "$MIR_SERVER_FILE"
6 fi
7
8+ initctl set-env --global MIR_SERVER_PROMPT_FILE=1
9+
10 initctl emit scope-ui-starting
11 initctl emit --no-wait indicator-services-start
12 end script
13@@ -53,4 +55,5 @@
14 post-stop script
15 initctl set-env --global MIR_SOCKET=$UNITY_MIR_SOCKET
16 initctl unset-env --global UNITY_MIR_SOCKET
17+ initctl unset-env --global MIR_SERVER_PROMPT_FILE
18 end script

Subscribers

People subscribed via source and target branches