unity-notifications

Merge lp://qastaging/~macslow/unity-notifications/fix-1256011 into lp://qastaging/unity-notifications

  1. fix-1256011
  2. Merge into trunk
Proposed by Mirco Müller
Status: Merged
Approved by: Michael Zanetti
Approved revision: 193
Merged at revision: 191
Proposed branch: lp://qastaging/~macslow/unity-notifications/fix-1256011
Merge into: lp://qastaging/unity-notifications
Diff against target: 41 lines (+14/-3)
2 files modified
debian/control (+1/-0)
src/CMakeLists.txt (+13/-3)
To merge this branch: bzr merge lp://qastaging/~macslow/unity-notifications/fix-1256011
Reviewer Review Type Date Requested Status
Michael Zanetti (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+197167@code.qastaging.launchpad.net

Commit message

Use new plugin-path provided by pkgconfig of libunity-api-dev, thus fixing LP #1256011.

Description of the change

Use new plugin-path provided by pkgconfig of libunity-api-dev, thus fixing LP #1256011.

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

FAILED: Continuous integration, rev:190
http://jenkins.qa.ubuntu.com/job/unity-notifications-ci/34/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-amd64-ci/9/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-armhf-ci/9/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-i386-ci/9/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-notifications-ci/34/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~macslow/unity-notifications/fix-1256011 updated
191. By Mirco Müller

libunity-api-dev needs to be added as a build-dependency too

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

FAILED: Continuous integration, rev:191
http://jenkins.qa.ubuntu.com/job/unity-notifications-ci/35/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-amd64-ci/10/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-armhf-ci/10/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-i386-ci/10/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-notifications-ci/35/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~macslow/unity-notifications/fix-1256011 updated
192. By Mirco Müller

*sigh*

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

PASSED: Continuous integration, rev:192
http://jenkins.qa.ubuntu.com/job/unity-notifications-ci/36/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-amd64-ci/11
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-armhf-ci/11
        deb: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-armhf-ci/11/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-i386-ci/11

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-notifications-ci/36/rebuild

review: Approve (continuous-integration)
lp://qastaging/~macslow/unity-notifications/fix-1256011 updated
193. By Mirco Müller

Merged with trunk and fixed conflicts.

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

PASSED: Continuous integration, rev:193
http://jenkins.qa.ubuntu.com/job/unity-notifications-ci/38/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-amd64-ci/13
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-armhf-ci/13
        deb: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-armhf-ci/13/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-notifications-trusty-i386-ci/13

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-notifications-ci/38/rebuild

review: Approve (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

ack

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: