Merge lp:~cjwatson/libdbusmenu/verbose-tests into lp:libdbusmenu/14.04

Proposed by Colin Watson
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 469
Merged at revision: 470
Proposed branch: lp:~cjwatson/libdbusmenu/verbose-tests
Merge into: lp:libdbusmenu/14.04
Diff against target: 15 lines (+2/-2)
1 file modified
debian/rules (+2/-2)
To merge this branch: bzr merge lp:~cjwatson/libdbusmenu/verbose-tests
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Approve
Charles Kerr (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+211346@code.launchpad.net

Commit message

Run tests with VERBOSE=1 so that Automake shows more detail.

Description of the change

Get Automake to print test logs on failure. This would be helpful in diagnosing build failures such as this:

  https://launchpad.net/ubuntu/+archive/test-rebuild-20140307/+build/5698825

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

Looks like this is failing due to bug #1294165, which is in progress w/ https://code.launchpad.net/~charlesk/libdbusmenu/lp-1294165-ftbfs

Revision history for this message
Charles Kerr (charlesk) wrote :

Other than that, LGTM

review: Approve
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/rules'
2--- debian/rules 2014-01-09 17:38:19 +0000
3+++ debian/rules 2014-03-17 15:53:25 +0000
4@@ -37,9 +37,9 @@
5 dotest-%:
6 ifeq ($(DEB_HOST_ARCH),ppc64el)
7 : # ignore the test results on $(DEB_HOST_ARCH)
8- -xvfb-run -a dh_auto_test --builddirectory=builddir/$*
9+ -VERBOSE=1 xvfb-run -a dh_auto_test --builddirectory=builddir/$*
10 else
11- xvfb-run -a dh_auto_test --builddirectory=builddir/$*
12+ VERBOSE=1 xvfb-run -a dh_auto_test --builddirectory=builddir/$*
13 endif
14
15 override_dh_install:

Subscribers

People subscribed via source and target branches

to all changes: