Merge into trunk : fix-1189281 : Code : Notify OSD

Status:

Merged

Approved by:

Sebastien Bacher on 2013-06-11

Approved revision:

466

Merged at revision:

466

Proposed branch:

lp://qastaging/~larsu/notify-osd/fix-1189281

Merge into:

lp://qastaging/notify-osd

Diff against target:

74 lines (+12/-6)

2 files modified

src/display.c (+6/-3)
src/stack.c (+6/-3)

To merge this branch:

bzr merge lp://qastaging/~larsu/notify-osd/fix-1189281

High

Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
PS Jenkins bot (community)	continuous-integration		Approve on 2013-06-11
Mirco Müller (community)			Approve on 2013-06-11
Sebastien Bacher		2013-06-10	Pending
Review via email: mp+168550@code.qastaging.launchpad.net

Commit message

Don't crash when an application is fullscreen when a notification arrives.

Description of the change

Don't crash when an application is fullscreen when a notification arrives.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-06-10:

#

FAILED: Continuous integration, rev:466
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/~larsu/notify-osd/fix-1189281/+merge/168550/+edit-commit-message

Click here to trigger a rebuild:
http://s-jenkins:8080/job/notify-osd-ci/2/rebuild

FAILED: Continuous integration, rev:466
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/~larsu/notify-osd/fix-1189281/+merge/168550/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/notify-osd-ci/2/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-amd64-ci/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-armhf-ci/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-i386-ci/1
Coverity artifacts:
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10932.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10933.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10934.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10935.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10936.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10937.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10938.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10939.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10940.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10941.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_13648.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_13649.html

Click here to trigger a rebuild:
http://s-jenkins:8080/job/notify-osd-ci/2/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

Mirco Müller (macslow) wrote on 2013-06-11:

#

Looks ok.

review: Approve

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-06-11:

#

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-06-11:

#

Click here to trigger a rebuild:
http://s-jenkins:8080/job/notify-osd-ci/3/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-06-11:

#

PASSED: Continuous integration, rev:466
http://jenkins.qa.ubuntu.com/job/notify-osd-ci/4/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-coverity-check/34
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-amd64-ci/3
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-armhf-ci/3
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-i386-ci/3
Coverity artifacts:
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/34/artifact/results/coverity/NO_COVERITY_DEFECTS.txt

Click here to trigger a rebuild:
http://s-jenkins:8080/job/notify-osd-ci/4/rebuild

review: Approve (continuous-integration)

Notify OSD

Merge lp://qastaging/~larsu/notify-osd/fix-1189281 into lp://qastaging/notify-osd

Commit message

Description of the change

Preview Diff

Subscribers