Merge lp:~charlesk/indicator-location/lp-1234975 into lp:indicator-location/13.10
Proposed by
Charles Kerr
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Ted Gould | ||||
Approved revision: | 52 | ||||
Merged at revision: | 51 | ||||
Proposed branch: | lp:~charlesk/indicator-location/lp-1234975 | ||||
Merge into: | lp:indicator-location/13.10 | ||||
Diff against target: |
26 lines (+3/-2) 2 files modified
src/phone.cc (+2/-1) tests/phone-test.cc (+1/-1) |
||||
To merge this branch: | bzr merge lp:~charlesk/indicator-location/lp-1234975 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ted Gould (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+189937@code.launchpad.net |
Description of the change
Since the "Location Settings" panel is disabled in system-settings for 13.10, hide the indicator's menuitem that activates that panel.
To post a comment you must log in.
PASSED: Continuous integration, rev:52 jenkins. qa.ubuntu. com/job/ indicator- location- ci/22/ jenkins. qa.ubuntu. com/job/ indicator- location- saucy-amd64- ci/22 jenkins. qa.ubuntu. com/job/ indicator- location- saucy-armhf- ci/21 jenkins. qa.ubuntu. com/job/ indicator- location- saucy-armhf- ci/21/artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: 10.97.0. 26:8080/ job/indicator- location- ci/22/rebuild
http://