Merge lp://staging/~charlesk/indicator-power/custom-bus-for-upower into lp://staging/indicator-power/15.04
Status: | Approved |
---|---|
Approved by: | Ted Gould |
Approved revision: | 283 |
Proposed branch: | lp://staging/~charlesk/indicator-power/custom-bus-for-upower |
Merge into: | lp://staging/indicator-power/15.04 |
Diff against target: |
1005 lines (+556/-176) 7 files modified
README (+11/-0) src/CMakeLists.txt (+1/-0) src/brightness.c (+71/-30) src/bus.c (+195/-0) src/bus.h (+50/-0) src/device-provider-upower.c (+67/-23) src/service.c (+161/-123) |
To merge this branch: | bzr merge lp://staging/~charlesk/indicator-power/custom-bus-for-upower |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ted Gould (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email:
|
Commit message
Add support for finer-grained dbus service mocking
Description of the change
== Change Description
Add support for finer-grained dbus service mocking.
Use case: say we want to feed the indicator a dbusmocked upower, but don't want to mock the entire system bus because we also talk to powerd on the system bus. This lets us specify via environment variables where to look for each bus dependency.
== Checklist
> Are there any related MPs required for this MP to build/function as expected? Please list.
No
> Is your branch in sync with latest trunk? (e.g. bzr pull lp:trunk -> no changes)
Yes
> Did the code build without warnings?
Yes
> Did the tests run successfully?
Yes
> Did you perform an exploratory manual test run of your code change and any related functionality?
Yes
> If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A
> Did your component test plan pass? If on a device, what image number?
15.03 nexus 4 r63
> Please list which manual tests are germane for the reviewer in this MR.
This is QA scaffolding; no manual tests affected
> Did you provide a link to this page https:/
Yes
Unmerged revisions
- 283. By Charles Kerr
-
pass the G_DBUS_
CONNECTION_ FLAGS_AUTHENTIC ATION_CLIENT flag when calling g_dbus_ connection_ new_for_ address( ) on an arbitrary bus address - 282. By Charles Kerr
-
when connecting to a custom bus address, use G_DBUS_
CONNECTION_ FLAGS_MESSAGE_ BUS_CONNECTION - 281. By Charles Kerr
-
document the debugging environment variables
- 280. By Charles Kerr
-
move indicator_
power_get_ screen_ busname( ) to bus.c - 279. By Charles Kerr
-
use indicator_
power_get_ bus() for acquiring the bus that we look for powerd on - 278. By Charles Kerr
-
use indicator_
power_get_ bus() for acquiring the bus that we look for upower on - 277. By Charles Kerr
-
use indicator_
power_get_ bus() for acquiring the indicator's bus - 276. By Charles Kerr
-
add indicator_
power_get_ bus() which lets environment variables let a custom bus override other buses.
PASSED: Continuous integration, rev:283 jenkins. qa.ubuntu. com/job/ indicator- power-ci/ 145/ jenkins. qa.ubuntu. com/job/ indicator- power-vivid- amd64-ci/ 4 jenkins. qa.ubuntu. com/job/ indicator- power-vivid- armhf-ci/ 4 jenkins. qa.ubuntu. com/job/ indicator- power-vivid- armhf-ci/ 4/artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/indicator- power-ci/ 145/rebuild
http://