Merge lp://staging/~allanlesage/kubuntu-packaging/qtlocation-enable-tests into lp://staging/~kubuntu-packagers/kubuntu-packaging/qtlocation-opensource-src

Proposed by Allan LeSage
Status: Merged
Merged at revision: 47
Proposed branch: lp://staging/~allanlesage/kubuntu-packaging/qtlocation-enable-tests
Merge into: lp://staging/~kubuntu-packagers/kubuntu-packaging/qtlocation-opensource-src
Diff against target: 269 lines (+216/-0)
6 files modified
debian/changelog (+9/-0)
debian/control (+1/-0)
debian/patches/enable_coverage_reporting.patch (+156/-0)
debian/patches/series (+2/-0)
debian/patches/skip_failing_tests.patch (+44/-0)
debian/rules (+4/-0)
To merge this branch: bzr merge lp://staging/~allanlesage/kubuntu-packaging/qtlocation-enable-tests
Reviewer Review Type Date Requested Status
Timo Jyrinki Approve
Review via email: mp+199367@code.staging.launchpad.net

Description of the change

Add tests to the Debian build; enable coverage reporting for Jenkins consumption; skip some failing tests.

Do we want a "make check" target at the root level of this project? The layout is unfamiliar to me, coming from autotools-type projects, however I don't know the common patterns of Qt projects, maybe this is standard.

To post a comment you must log in.
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :
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