Merge lp:~brian.curtin/ubuntu-sso-client/networkstates-update into lp:ubuntu-sso-client

Proposed by Brian Curtin on 2012-10-30
Status: Merged
Approved by: Brian Curtin on 2012-10-30
Approved revision: 1012
Merged at revision: 1011
Proposed branch: lp:~brian.curtin/ubuntu-sso-client/networkstates-update
Merge into: lp:ubuntu-sso-client
Diff against target: 21 lines (+1/-3)
1 file modified
ubuntu_sso/networkstate/windows.py (+1/-3)
To merge this branch: bzr merge lp:~brian.curtin/ubuntu-sso-client/networkstates-update
Reviewer Review Type Date Requested Status
Michał Karnicki (community) Approve on 2012-10-30
dobey (community) 2012-10-30 Approve on 2012-10-30
Review via email: mp+132206@code.launchpad.net

Commit Message

- Use states from networkstates instead of re-defining them locally.

Description of the Change

Change Windows-specific code to follow the other platforms by using constants from networkstates rather than defining their own. This fixes ubuntuone-client failures on Windows.

To post a comment you must log in.
dobey (dobey) :
review: Approve
Ubuntu One Auto Pilot (otto-pilot) wrote :
Download full text (89.5 KiB)

The attempt to merge lp:~brian.curtin/ubuntu-sso-client/networkstates-update into lp:ubuntu-sso-client failed. Below is the output from the failed tests.

*** Running QT test suite for ubuntu_sso ***
running build
Compiled data/qt/proxy_credentials_dialog.ui into ubuntu_sso/qt/ui/proxy_credentials_dialog_ui.py
compiled data/qt/resources.qrc into ubuntu_sso/qt/ui/resources_rc.py
Compiled data/qt/setup_account.ui into ubuntu_sso/qt/ui/setup_account_ui.py
Compiled data/qt/reset_password.ui into ubuntu_sso/qt/ui/reset_password_ui.py
Compiled data/qt/network_detection.ui into ubuntu_sso/qt/ui/network_detection_ui.py
Compiled data/qt/forgotten_password.ui into ubuntu_sso/qt/ui/forgotten_password_ui.py
Compiled data/qt/ssl_dialog.ui into ubuntu_sso/qt/ui/ssl_dialog_ui.py
Compiled data/qt/success_message.ui into ubuntu_sso/qt/ui/success_message_ui.py
Compiled data/qt/loadingoverlay.ui into ubuntu_sso/qt/ui/loadingoverlay_ui.py
Compiled data/qt/current_user_sign_in.ui into ubuntu_sso/qt/ui/current_user_sign_in_ui.py
Compiled data/qt/error_message.ui into ubuntu_sso/qt/ui/error_message_ui.py
Compiled data/qt/email_verification.ui into ubuntu_sso/qt/ui/email_verification_ui.py
running build_py
creating build
creating build/lib.linux-x86_64-2.7
creating build/lib.linux-x86_64-2.7/ubuntu_sso
copying ubuntu_sso/__init__.py -> build/lib.linux-x86_64-2.7/ubuntu_sso
copying ubuntu_sso/account.py -> build/lib.linux-x86_64-2.7/ubuntu_sso
copying ubuntu_sso/credentials.py -> build/lib.linux-x86_64-2.7/ubuntu_sso
copying ubuntu_sso/logger.py -> build/lib.linux-x86_64-2.7/ubuntu_sso
creating build/lib.linux-x86_64-2.7/ubuntu_sso/tests
copying ubuntu_sso/tests/__init__.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/tests
copying ubuntu_sso/tests/linux.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/tests
copying ubuntu_sso/tests/test_credentials.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/tests
copying ubuntu_sso/tests/test_account.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/tests
creating build/lib.linux-x86_64-2.7/ubuntu_sso/keyring
copying ubuntu_sso/keyring/__init__.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/keyring
copying ubuntu_sso/keyring/linux.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/keyring
copying ubuntu_sso/keyring/pykeyring.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/keyring
creating build/lib.linux-x86_64-2.7/ubuntu_sso/keyring/tests
copying ubuntu_sso/keyring/tests/__init__.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/keyring/tests
copying ubuntu_sso/keyring/tests/test_common.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/keyring/tests
copying ubuntu_sso/keyring/tests/test_linux.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/keyring/tests
copying ubuntu_sso/keyring/tests/test_pykeyring.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/keyring/tests
creating build/lib.linux-x86_64-2.7/ubuntu_sso/main
copying ubuntu_sso/main/__init__.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/main
copying ubuntu_sso/main/glib.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/main
copying ubuntu_sso/main/linux.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/main
copying ubuntu_sso/main/darwin.py -> build/lib.linux-x86_64-2.7/ubuntu_sso/main
copying ubuntu_sso/main/windows.py -> build/lib.linux-...

Michał Karnicki (karni) :
review: Approve
1012. By Brian Curtin on 2012-10-30

Remove unused UNKNOWN name

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntu_sso/networkstate/windows.py'
2--- ubuntu_sso/networkstate/windows.py 2012-06-22 16:12:03 +0000
3+++ ubuntu_sso/networkstate/windows.py 2012-10-30 22:07:20 +0000
4@@ -45,6 +45,7 @@
5 # pylint: enable=F0401
6
7 from ubuntu_sso.networkstate import NetworkFailException
8+from ubuntu_sso.networkstate.networkstates import ONLINE, OFFLINE
9 from ubuntu_sso.logger import setup_logging
10
11 logger = setup_logging("ubuntu_sso.networkstate")
12@@ -53,9 +54,6 @@
13 # it clear for later developers.
14 # pylint: disable=C0103
15
16-# Values returned by the callback
17-ONLINE, OFFLINE, UNKNOWN = object(), object(), object()
18-
19 ## from EventSys.h
20 PROGID_EventSystem = "EventSystem.EventSystem"
21 PROGID_EventSubscription = "EventSystem.EventSubscription"

Subscribers

People subscribed via source and target branches