Merge lp:~laney/ubuntu-system-settings/upower0.99 into lp:ubuntu-system-settings

Proposed by Iain Lane
Status: Merged
Approved by: Ken VanDine
Approved revision: 912
Merged at revision: 1170
Proposed branch: lp:~laney/ubuntu-system-settings/upower0.99
Merge into: lp:ubuntu-system-settings
Diff against target: 42 lines (+8/-2)
2 files modified
plugins/battery/battery.cpp (+4/-1)
plugins/battery/plugin/battery-plugin.cpp (+4/-1)
To merge this branch: bzr merge lp:~laney/ubuntu-system-settings/upower0.99
Reviewer Review Type Date Requested Status
Ken VanDine Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+230988@code.launchpad.net

Commit message

Update to work with UPower >= 0.99 as well as current versions

Description of the change

Mainly I want the CI debs but also I hope this is right already.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:910
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-ci/1235/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/3607
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/2827
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-utopic-amd64-ci/428
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-utopic-armhf-ci/424
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-utopic-armhf-ci/424/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-utopic-i386-ci/427
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/3494
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/4854
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/4854/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/11554
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/2292
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/3111
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/3111/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-ci/1235/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Iain Lane (laney) wrote :

Seems okay to me, & tests pass with 0.99 - please review.

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

On Fri, Sep 12, 2014 at 03:39:02PM -0000, PS Jenkins bot wrote:
> Review: Needs Fixing continuous-integration
>
> FAILED: Continuous integration, rev:910

ffs

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

911. By Iain Lane

Merge trunk

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
912. By Iain Lane

Merge trunk

Revision history for this message
Iain Lane (laney) wrote :

I just re-merged trunk, no conflicts

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

Nice to have this ready, fixes the FTBFS in vivid :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/battery/battery.cpp'
--- plugins/battery/battery.cpp 2014-08-28 07:22:28 +0000
+++ plugins/battery/battery.cpp 2014-10-30 16:41:15 +0000
@@ -63,15 +63,18 @@
6363
64void Battery::buildDeviceString() {64void Battery::buildDeviceString() {
65 UpClient *client;65 UpClient *client;
66 gboolean returnIsOk;
67 GPtrArray *devices;66 GPtrArray *devices;
68 UpDeviceKind kind;67 UpDeviceKind kind;
6968
69#if !UP_CHECK_VERSION(0, 99, 0)
70 gboolean returnIsOk;
71
70 client = up_client_new();72 client = up_client_new();
71 returnIsOk = up_client_enumerate_devices_sync(client, nullptr, nullptr);73 returnIsOk = up_client_enumerate_devices_sync(client, nullptr, nullptr);
7274
73 if(!returnIsOk)75 if(!returnIsOk)
74 return;76 return;
77#endif
7578
76 devices = up_client_get_devices(client);79 devices = up_client_get_devices(client);
7780
7881
=== modified file 'plugins/battery/plugin/battery-plugin.cpp'
--- plugins/battery/plugin/battery-plugin.cpp 2014-07-23 13:37:06 +0000
+++ plugins/battery/plugin/battery-plugin.cpp 2014-10-30 16:41:15 +0000
@@ -49,10 +49,13 @@
49{49{
50 BatteryItem *item (static_cast<BatteryItem *> (user_data));50 BatteryItem *item (static_cast<BatteryItem *> (user_data));
5151
52#if !UP_CHECK_VERSION(0, 99, 0)
52 gboolean ret = up_client_enumerate_devices_sync (client, nullptr, nullptr);53 gboolean ret = up_client_enumerate_devices_sync (client, nullptr, nullptr);
53 if (!ret) {54 if (!ret) {
54 item->setVisibility (false);55 item->setVisibility (false);
55 } else {56 } else
57#endif
58 {
56 GPtrArray *devices = up_client_get_devices (client);59 GPtrArray *devices = up_client_get_devices (client);
57 item->setVisibility (devices->len > 0);60 item->setVisibility (devices->len > 0);
58 g_ptr_array_unref (devices);61 g_ptr_array_unref (devices);

Subscribers

People subscribed via source and target branches