libappindicator

Merge lp://qastaging/~3v1n0/libappindicator/rm-approver into lp://qastaging/libappindicator/13.10

  1. rm-approver
  2. Merge into trunk.13.10
Proposed by Ted Gould
Status: Merged
Approved by: Ted Gould
Approved revision: 247
Merged at revision: 264
Proposed branch: lp://qastaging/~3v1n0/libappindicator/rm-approver
Merge into: lp://qastaging/libappindicator/13.10
Prerequisite: lp://qastaging/~ted/libappindicator/saucy-fix
Diff against target: 17 lines (+0/-7)
1 file modified
src/notification-watcher.xml (+0/-7)
To merge this branch: bzr merge lp://qastaging/~3v1n0/libappindicator/rm-approver
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Ted Gould (community) Approve
Review via email: mp+168287@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2013-06-07.

Commit message

NotificationWatcher interface has not anymore the XAyatanaRegisterNotificationApprover method

Description of the change

NotificationWatcher interface has not anymore the XAyatanaRegisterNotificationApprover method

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

FAILED: Continuous integration, rev:247
http://jenkins.qa.ubuntu.com/job/libappindicator-ci/16/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/libappindicator-ci/./build=pbuilder,distribution=coverity-raring,flavor=amd64/16/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/libappindicator-ci/./build=pbuilder,distribution=raring,flavor=i386/16/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/libappindicator-ci/16//rebuild/?

review: Needs Fixing (continuous-integration)
Revision history for this message
Ted Gould (ted) : Posted in a previous version of this proposal
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/libappindicator-autolanding/5/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/libappindicator-autolanding/./build=pbuilder,distribution=coverity-raring,flavor=amd64/5/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/libappindicator-autolanding/./build=pbuilder,distribution=raring,flavor=i386/5/console

review: Needs Fixing (continuous-integration)
Revision history for this message
Allan LeSage (allanlesage) wrote : Posted in a previous version of this proposal

I'm able to show a clean build in quantal but not in raring, FWIW--we might need some packaging help to land this.

Revision history for this message
Ted Gould (ted) : Posted in a previous version of this proposal
review: Approve
Revision history for this message
Ted Gould (ted) wrote : Posted in a previous version of this proposal

Resubmitting to current trunk.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/libappindicator-autolanding/5/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/libappindicator-saucy-amd64-autolanding/1/console

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

FAILED: Continuous integration, rev:247
http://jenkins.qa.ubuntu.com/job/libappindicator-ci/12/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/libappindicator-saucy-amd64-ci/2/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/libappindicator-ci/12/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Ted Gould (ted) wrote :

Rebasing on Saucy fix to see if we can get Jenkins happy.

Revision history for this message
Ted Gould (ted) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

> Status: Approved => Merged

Oh, finally... ^_^

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