Merge lp://staging/~phablet-team/telephony-service/fix_qtdbus_5.6 into lp://staging/telephony-service/staging

Proposed by Gustavo Pichorim Boiko
Status: Merged
Approved by: Roberto Mier Escandon
Approved revision: 1206
Merged at revision: 1228
Proposed branch: lp://staging/~phablet-team/telephony-service/fix_qtdbus_5.6
Merge into: lp://staging/telephony-service/staging
Prerequisite: lp://staging/~phablet-team/telephony-service/re-enable_tests
Diff against target: 84 lines (+7/-24)
4 files modified
approver/approverdbus.cpp (+1/-6)
handler/handlerdbus.cpp (+1/-6)
indicator/indicatordbus.cpp (+1/-6)
tests/common/mock/mockconnectiondbus.cpp (+4/-6)
To merge this branch: bzr merge lp://staging/~phablet-team/telephony-service/fix_qtdbus_5.6
Reviewer Review Type Date Requested Status
Roberto Mier Escandon (community) Approve
system-apps-ci-bot continuous-integration Pending
Review via email: mp+309396@code.staging.launchpad.net

This proposal supersedes a proposal from 2016-09-19.

Commit message

Adapt the registration of objects and services to the way QtDBus works from 5.6.x on.

Description of the change

Adapt the registration of objects and services to the way QtDBus works from 5.6.x on.

To post a comment you must log in.
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote : Posted in a previous version of this proposal

FAILED: Continuous integration, rev:1205
https://jenkins.canonical.com/system-apps/job/lp-telephony-service-ci/60/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/1563/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/1563
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1410
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1410/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1410
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1410/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1410
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1410/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1410
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1410/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1410
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1410/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1410
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1410/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1410/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1410
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1410/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1410/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-telephony-service-ci/60/rebuild

review: Needs Fixing (continuous-integration)
1206. By Gustavo Pichorim Boiko

Merge parent branch

Revision history for this message
Roberto Mier Escandon (rmescandon) wrote :

lgtm

review: Approve

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

to all changes: