Unity 8

Merge lp:~aacid/unity8/less_repeated_stuff_cmake into lp:unity8

  1. less_repeated_stuff_cmake
  2. Merge into trunk
Proposed by Albert Astals Cid
Status: Merged
Approved by: Michael Zanetti
Approved revision: 187
Merged at revision: 188
Proposed branch: lp:~aacid/unity8/less_repeated_stuff_cmake
Merge into: lp:unity8
Diff against target: 82 lines (+1/-26)
6 files modified
CMakeLists.txt (+1/-0)
plugins/Ubuntu/Gestures/CMakeLists.txt (+0/-6)
tests/mocks/HudClient/CMakeLists.txt (+0/-5)
tests/mocks/Ubuntu/Application/CMakeLists.txt (+0/-6)
tests/mocks/Unity/CMakeLists.txt (+0/-5)
tests/utils/modules/Unity/Test/CMakeLists.txt (+0/-4)
To merge this branch: bzr merge lp:~aacid/unity8/less_repeated_stuff_cmake
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Michael Zanetti (community) Approve
Review via email: mp+179875@code.launchpad.net

Commit message

Remove unneeded stuff from CMakelists.txt

set(CMAKE_AUTOMOC ON)

include(FindPkgConfig)
find_package(Qt5Core REQUIRED)
find_package(Qt5Quick REQUIRED)

Are already on the top-level CMakeLists.txt so no need to write them again
Well, actually the Qt5Core wasn't and i added it, it's not really necessary
since it's pulled by the others that depend on it, but it doesn't hurt to be
explicit

To post a comment you must log in.
Revision history for this message
Michael Zanetti (mzanetti) wrote :

still builds fine for me. As we agreed that plugins must not be self-contained at this point in time, I'll approve

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

PASSED: Continuous integration, rev:187
http://jenkins.qa.ubuntu.com/job/unity8-ci/584/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/2139
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-saucy/1028
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-saucy-armhf-ci/585
        deb: http://jenkins.qa.ubuntu.com/job/unity8-saucy-armhf-ci/585/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-saucy-i386-ci/584
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy/2145
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy/2145/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/1822

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity8-ci/584/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2013-07-19 16:22:50 +0000
3+++ CMakeLists.txt 2013-08-13 09:14:35 +0000
4@@ -48,6 +48,7 @@
5 ${CMAKE_SOURCE_DIR}/src ${CMAKE_SOURCE_DIR}/tests)
6
7 include(FindPkgConfig)
8+find_package(Qt5Core)
9 find_package(Qt5Qml)
10 find_package(Qt5Quick)
11 find_package(Qt5Gui)
12
13=== modified file 'plugins/Ubuntu/Gestures/CMakeLists.txt'
14--- plugins/Ubuntu/Gestures/CMakeLists.txt 2013-07-08 15:12:28 +0000
15+++ plugins/Ubuntu/Gestures/CMakeLists.txt 2013-08-13 09:14:35 +0000
16@@ -1,12 +1,6 @@
17 # export_qmlplugin macro
18 include(Plugins)
19
20-set(CMAKE_AUTOMOC ON)
21-
22-include(FindPkgConfig)
23-find_package(Qt5Core REQUIRED)
24-find_package(Qt5Quick REQUIRED)
25-
26 set(UbuntuGestureQml_SOURCES
27 plugin.cpp
28 AxisVelocityCalculator.cpp
29
30=== modified file 'tests/mocks/HudClient/CMakeLists.txt'
31--- tests/mocks/HudClient/CMakeLists.txt 2013-06-05 22:03:08 +0000
32+++ tests/mocks/HudClient/CMakeLists.txt 2013-08-13 09:14:35 +0000
33@@ -1,8 +1,3 @@
34-set(CMAKE_AUTOMOC ON)
35-
36-include(FindPkgConfig)
37-find_package(Qt5Core REQUIRED)
38-find_package(Qt5Quick REQUIRED)
39 pkg_search_module(DEE dee-1.0 REQUIRED)
40 pkg_search_module(GOBJECT gobject-2.0 REQUIRED)
41 pkg_search_module(HUDCLIENT hud-client-2 REQUIRED)
42
43=== modified file 'tests/mocks/Ubuntu/Application/CMakeLists.txt'
44--- tests/mocks/Ubuntu/Application/CMakeLists.txt 2013-06-26 10:15:58 +0000
45+++ tests/mocks/Ubuntu/Application/CMakeLists.txt 2013-08-13 09:14:35 +0000
46@@ -1,9 +1,3 @@
47-set(CMAKE_AUTOMOC ON)
48-
49-include(FindPkgConfig)
50-find_package(Qt5Core REQUIRED)
51-find_package(Qt5Quick REQUIRED)
52-
53 set(FakeUbuntuApplicationQml_SOURCES
54 plugin.cpp
55 ApplicationInfo.cpp
56
57=== modified file 'tests/mocks/Unity/CMakeLists.txt'
58--- tests/mocks/Unity/CMakeLists.txt 2013-07-26 11:10:55 +0000
59+++ tests/mocks/Unity/CMakeLists.txt 2013-08-13 09:14:35 +0000
60@@ -1,10 +1,5 @@
61-set(CMAKE_AUTOMOC ON)
62-
63 add_subdirectory(Indicators)
64
65-include(FindPkgConfig)
66-find_package(Qt5Core REQUIRED)
67-find_package(Qt5Quick REQUIRED)
68 pkg_search_module(DEE dee-1.0 REQUIRED)
69 pkg_search_module(GOBJECT gobject-2.0 REQUIRED)
70 pkg_search_module(DEEQT libdee-qt5 REQUIRED)
71
72=== modified file 'tests/utils/modules/Unity/Test/CMakeLists.txt'
73--- tests/utils/modules/Unity/Test/CMakeLists.txt 2013-06-05 22:03:08 +0000
74+++ tests/utils/modules/Unity/Test/CMakeLists.txt 2013-08-13 09:14:35 +0000
75@@ -1,7 +1,3 @@
76-find_package(Qt5Core REQUIRED)
77-find_package(Qt5Quick REQUIRED)
78-
79-set(CMAKE_AUTOMOC ON)
80 add_definitions(-DQT_NO_KEYWORDS)
81
82 set(UnityTestQML_SOURCES

Subscribers

People subscribed via source and target branches