Merge lp:~dandrader/qtmir/optionalTestsBuild into lp:qtmir

Proposed by Daniel d'Andrada
Status: Merged
Approved by: Gerry Boland
Approved revision: 255
Merged at revision: 259
Proposed branch: lp:~dandrader/qtmir/optionalTestsBuild
Merge into: lp:qtmir
Diff against target: 27 lines (+8/-3)
2 files modified
README (+2/-1)
qtmir.pro (+6/-2)
To merge this branch: bzr merge lp:~dandrader/qtmir/optionalTestsBuild
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Gerry Boland (community) Approve
Review via email: mp+235467@code.launchpad.net

Commit message

CONFIG+=no_tests to avoid building the tests and therefore speed up the build

Description of the change

* Are there any related MPs required for this MP to build/function as expected? Please list.
No

* Did you perform an exploratory manual test run of your code change and any related functionality?
Yes

* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
Not applicable

To post a comment you must log in.
Revision history for this message
Gerry Boland (gerboland) wrote :

LGTM

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'README'
2--- README 2014-08-28 16:25:07 +0000
3+++ README 2014-09-22 14:30:06 +0000
4@@ -12,7 +12,8 @@
5 $ qmake CONFIG+=debug
6 (is good to "make clean" beforehand)
7
8-
9+To avoid building the tests (to speed up the build):
10+$ qmake CONFIG+=no_tests
11
12 To test, enter the demos directory, run:
13
14
15=== modified file 'qtmir.pro'
16--- qtmir.pro 2014-07-01 13:38:06 +0000
17+++ qtmir.pro 2014-09-22 14:30:06 +0000
18@@ -1,3 +1,7 @@
19 TEMPLATE = subdirs
20-SUBDIRS = src tests
21-tests.depends = src
22+SUBDIRS = src
23+
24+!no_tests {
25+ SUBDIRS += tests
26+ tests.depends = src
27+}

Subscribers

People subscribed via source and target branches