Unity Control Center

Merge lp://staging/~diwic/unity-control-center/lp1367693 into lp://staging/unity-control-center

  1. lp1367693
  2. Merge into trunk
Proposed by David Henningsson
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 12788
Merged at revision: 12789
Proposed branch: lp://staging/~diwic/unity-control-center/lp1367693
Merge into: lp://staging/unity-control-center
Diff against target: 60 lines (+12/-9)
2 files modified
panels/sound/gvc-mixer-control.c (+3/-3)
panels/sound/gvc-mixer-dialog.c (+9/-6)
To merge this branch: bzr merge lp://staging/~diwic/unity-control-center/lp1367693
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Robert Ancell Approve
Sebastien Bacher Pending
Review via email: mp+234816@code.staging.launchpad.net

Commit message

Make sure selecting a bluetooth input works even when the headset is in A2DP mode, and make sure the correct levels are shown in the level bar (Bug 1367693)

Description of the change

Hi Sebastian!

I've been asked (by oem-priority) to fix bug 1367693 and SRU it into Trusty. I've come up with fixes for the bug (which turned out to be two bugs in one), but can you review my fixes and help me with the process, i e what else do I need to do (in form of testing etc) to make sure this gets SRUed into 14.04?

So far I've just been running testing my fixes with lp:unity-control-center's trunk on 14.04.

Thanks!

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

FAILED: Continuous integration, rev:12788
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~diwic/unity-control-center/lp1367693/+merge/234816/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity-control-center-ci/112/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-amd64-ci/14
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-armhf-ci/14
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-i386-ci/14

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-control-center-ci/112/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Ancell (robert-ancell) wrote :

I don't know the code in enough detail to know the implications of the change but they make sense to me. You should file a bug against libgnome-volume-control as their code appears to contain the same error. They're also more likely to give a better review.

Please update the commit message to be more informative of the problem and solution.

To propose this into trusty make a merge proposal to lp:unity-control-center/14.04

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:12788
http://jenkins.qa.ubuntu.com/job/unity-control-center-ci/113/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-amd64-ci/15
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-armhf-ci/15
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-i386-ci/15

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-control-center-ci/113/rebuild

review: Approve (continuous-integration)
Revision history for this message
David Henningsson (diwic) wrote :

Hi Robert and thanks for the review,

> I don't know the code in enough detail to know the implications of the change
> but they make sense to me. You should file a bug against libgnome-volume-
> control as their code appears to contain the same error. They're also more
> likely to give a better review.

The first bug has been upstreamed to:

 https://bugzilla.gnome.org/show_bug.cgi?id=736943

The second bug is in gvc-mixer-dialog and their gvc-mixer-dialog looks slightly different, so the fix could also be slightly different. And after all it's mostly a band-aid for a PulseAudio oddity, so they may argue that things should be fixed in PA instead (although such a behaviour change would be more difficult and nothing I would want to SRU into trusty).

> Please update the commit message to be more informative of the problem and
> solution.

Done.

> To propose this into trusty make a merge proposal to lp:unity-control-
> center/14.04

Done: https://code.launchpad.net/~diwic/unity-control-center/lp1367693-14.04/+merge/235243

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches