Merge lp://qastaging/~mterry/notify-osd/inline2 into lp://qastaging/~canonical-dx-team/notify-osd/precise

Proposed by Michael Terry
Status: Merged
Approved by: Allan LeSage
Approved revision: 459
Merged at revision: 459
Proposed branch: lp://qastaging/~mterry/notify-osd/inline2
Merge into: lp://qastaging/~canonical-dx-team/notify-osd/precise
Diff against target: 1006 lines (+927/-1)
14 files modified
.bzr-builddeb/default.conf (+2/-0)
debian/changelog (+683/-0)
debian/compat (+1/-0)
debian/control (+35/-0)
debian/copyright (+49/-0)
debian/notify-osd.install (+1/-0)
debian/notify-osd.postinst (+10/-0)
debian/notify-osd.postrm (+20/-0)
debian/notify-osd.preinst (+23/-0)
debian/rules (+13/-0)
debian/source_notify-osd.py (+83/-0)
debian/tests/control (+3/-0)
debian/tests/upstream (+2/-0)
tests/test-withlib.c (+2/-1)
To merge this branch: bzr merge lp://qastaging/~mterry/notify-osd/inline2
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Didier Roche-Tolomelli Approve
Review via email: mp+136258@code.qastaging.launchpad.net

Description of the change

This branch is based off lp:~robru/notify-osd/inline-packaging, with the following differences:
* raring->UNRELEASED
* Use ./autogen.sh during autoreconf
* Doesn't try to run tests during build, but instead as a dep8 test (needs its schema to be installed in system)
* I disabled a test that required the user to click a button in a dialog. The test suite should be automatic.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Allan LeSage (allanlesage) wrote :

Please co-ordinate with alesage when you want to land/approve, and I'll prepare the Jenkins job :) .

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Looks good to me, seems the jenkins job has been changed by Allan. Approving then :)

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

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