Merge lp:~sylvain-pineau/checkbox/create_connection_fix_vivid into lp:checkbox

Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 3825
Merged at revision: 3826
Proposed branch: lp:~sylvain-pineau/checkbox/create_connection_fix_vivid
Merge into: lp:checkbox
Diff against target: 11 lines (+1/-0)
1 file modified
providers/plainbox-provider-checkbox/bin/create_connection (+1/-0)
To merge this branch: bzr merge lp:~sylvain-pineau/checkbox/create_connection_fix_vivid
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+261423@code.launchpad.net

Description of the change

A fix for the create_connection script (for Vivid). Quoting the nmcli man page:

NetworkManager does not monitor changes to connection files by default. So we
need to use this command in order to tell NetworkManager to re-read the
connection profiles from disk when a change was made to them.

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

self-approving this one.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'providers/plainbox-provider-checkbox/bin/create_connection'
--- providers/plainbox-provider-checkbox/bin/create_connection 2015-06-02 08:22:55 +0000
+++ providers/plainbox-provider-checkbox/bin/create_connection 2015-06-08 16:09:05 +0000
@@ -137,6 +137,7 @@
137 stderr=DEVNULL,137 stderr=DEVNULL,
138 universal_newlines=True)138 universal_newlines=True)
139 except CalledProcessError:139 except CalledProcessError:
140 check_call(['nmcli', 'con', 'reload'])
140 nmcli_con_list = check_output(['nmcli', 'con', 'show'],141 nmcli_con_list = check_output(['nmcli', 'con', 'show'],
141 stderr=DEVNULL,142 stderr=DEVNULL,
142 universal_newlines=True)143 universal_newlines=True)

Subscribers

People subscribed via source and target branches