Telepathy Indicator

Merge lp://staging/~larsu/telepathy-indicator/watch-empathy into lp://staging/telepathy-indicator

  1. watch-empathy
  2. Merge into trunk
Proposed by Lars Karlitski
Status: Merged
Approved by: Ted Gould
Approved revision: 73
Merged at revision: 73
Proposed branch: lp://staging/~larsu/telepathy-indicator/watch-empathy
Merge into: lp://staging/telepathy-indicator
Prerequisite: lp://staging/~larsu/telepathy-indicator/gracefully-exit
Diff against target: 169 lines (+49/-28)
1 file modified
src/indicator-approver.c (+49/-28)
To merge this branch: bzr merge lp://staging/~larsu/telepathy-indicator/watch-empathy
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Ken VanDine Pending
Review via email: mp+149432@code.staging.launchpad.net

Commit message

Mark empathy as running only when it is actually running.

Description of the change

Mark empathy as running only when it is actually running.

This doesn't actually work yet due to a bug in indicator-messages. Will fix that shortly.

To post a comment you must log in.
lp://staging/~larsu/telepathy-indicator/watch-empathy updated
71. By Lars Karlitski

Quit when another instance is already running

72. By Lars Karlitski

Unref tp_am in the function it was created in

73. By Lars Karlitski

Update license header

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

PASSED: Continuous integration, rev:70
http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/5/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=coverity-raring,flavor=amd64/5/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=raring,flavor=armhf/5/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=raring,flavor=i386/5/console
Coverity artifacts:
    http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=coverity-raring,flavor=amd64/5//artifact/results/coverity/NO_COVERITY_DEFECTS.txt
    http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=raring,flavor=armhf/5//artifact/results/coverity/NO_COVERITY_DEFECTS.txt
    http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=raring,flavor=i386/5//artifact/results/coverity/NO_COVERITY_DEFECTS.txt

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/5//rebuild/?

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

PASSED: Continuous integration, rev:73
http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/6/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=coverity-raring,flavor=amd64/6/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=raring,flavor=armhf/6/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=raring,flavor=i386/6/console
Coverity artifacts:
    http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=coverity-raring,flavor=amd64/6//artifact/results/coverity/NO_COVERITY_DEFECTS.txt
    http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=raring,flavor=armhf/6//artifact/results/coverity/NO_COVERITY_DEFECTS.txt
    http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/./build=pbuilder,distribution=raring,flavor=i386/6//artifact/results/coverity/NO_COVERITY_DEFECTS.txt

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/telepathy-indicator-ci/6//rebuild/?

review: Approve (continuous-integration)
Revision history for this message
Ted Gould (ted) :
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: