Merge lp:~cszikszoy/do/nm-connectivity-fix into lp:do
Proposed by
Chris S.
Status: | Merged |
---|---|
Approved by: | Alex Launi |
Approved revision: | 1306 |
Merged at revision: | not available |
Proposed branch: | lp:~cszikszoy/do/nm-connectivity-fix |
Merge into: | lp:do |
Diff against target: |
21 lines 1 file modified
Do.Platform.Linux/src/Do.Platform/Do.Platform.Linux/NetworkService.cs (+4/-3) |
To merge this branch: | bzr merge lp:~cszikszoy/do/nm-connectivity-fix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alex Launi (community) | Approve | ||
Review via email: mp+12851@code.launchpad.net |
To post a comment you must log in.
This should fix the cases where people don't have network manaager. What's changed is that we set the connection state if we can find the network manager dbus. If there was an error (the nm dbus wasn't found), we log the error and just assume that we are connected. This way any plugins that query IsConnected won't fail when NM couldn't be found.