Merge lp:~ubuntu-clock-dev/ubuntu-clock-app/switch-to-set-cmake into lp:ubuntu-clock-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Approved by: Bartosz Kosiorek
Approved revision: 325
Merged at revision: 324
Proposed branch: lp:~ubuntu-clock-dev/ubuntu-clock-app/switch-to-set-cmake
Merge into: lp:ubuntu-clock-app
Diff against target: 111 lines (+54/-7)
6 files modified
app/alarm/CMakeLists.txt (+12/-1)
app/clock/CMakeLists.txt (+4/-1)
app/components/CMakeLists.txt (+13/-1)
app/graphics/CMakeLists.txt (+11/-2)
app/upstreamcomponents/CMakeLists.txt (+8/-1)
app/worldclock/CMakeLists.txt (+6/-1)
To merge this branch: bzr merge lp:~ubuntu-clock-dev/ubuntu-clock-app/switch-to-set-cmake
Reviewer Review Type Date Requested Status
Bartosz Kosiorek Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Nekhelesh Ramananthan Abstain
Review via email: mp+267677@code.launchpad.net

Commit message

Switched GLOB to Set in the cmake files

Description of the change

Switched GLOB to Set in the cmake files.

To post a comment you must log in.
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

The Graphics directory still hasn't been switched to Set(). @Bartosz can you do that. Somehow the files are not shown in Qtcreator.

review: Needs Fixing
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
325. By Nekhelesh Ramananthan

Fixed Graphics file cmake

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Fixed the graphics file not showing.

review: Abstain
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Bartosz Kosiorek (gang65) wrote :

It is working perfectly for me.
Thanks Nekhelesh.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app/alarm/CMakeLists.txt'
2--- app/alarm/CMakeLists.txt 2014-08-07 20:47:53 +0000
3+++ app/alarm/CMakeLists.txt 2015-08-11 16:05:23 +0000
4@@ -1,4 +1,15 @@
5-file(GLOB ALARM_QML_JS_FILES *.qml *.js)
6+set(ALARM_QML_JS_FILES
7+ AlarmDelegate.qml
8+ AlarmLabel.qml
9+ AlarmList.qml
10+ AlarmModelComponent.qml
11+ AlarmPage.qml
12+ AlarmRepeat.qml
13+ AlarmSettingsPage.qml
14+ AlarmSound.qml
15+ AlarmUtils.qml
16+ EditAlarmPage.qml
17+)
18
19 # make the files visible in the qtcreator tree
20 add_custom_target(ubuntu-clock-app_alarm_QMlFiles ALL SOURCES ${ALARM_QML_JS_FILES})
21
22=== modified file 'app/clock/CMakeLists.txt'
23--- app/clock/CMakeLists.txt 2014-08-07 20:47:53 +0000
24+++ app/clock/CMakeLists.txt 2015-08-11 16:05:23 +0000
25@@ -1,4 +1,7 @@
26-file(GLOB CLOCK_QML_JS_FILES *.qml *.js)
27+set(CLOCK_QML_JS_FILES
28+ ClockPage.qml
29+ MainClock.qml
30+)
31
32 # make the files visible in the qtcreator tree
33 add_custom_target(ubuntu-clock-app_clock_QMlFiles ALL SOURCES ${CLOCK_QML_JS_FILES})
34
35=== modified file 'app/components/CMakeLists.txt'
36--- app/components/CMakeLists.txt 2014-08-07 20:47:53 +0000
37+++ app/components/CMakeLists.txt 2015-08-11 16:05:23 +0000
38@@ -1,4 +1,16 @@
39-file(GLOB COMPONENTS_QML_JS_FILES *.qml *.js)
40+set(COMPONENTS_QML_JS_FILES
41+ AnalogMode.qml
42+ AnalogShadow.qml
43+ Background.qml
44+ Clock.qml
45+ ClockCircle.qml
46+ DigitalMode.qml
47+ DigitalShadow.qml
48+ EmptyState.qml
49+ ExpandableListItem.qml
50+ Shadow.qml
51+ SubtitledListItem.qml
52+)
53
54 # make the files visible in the qtcreator tree
55 add_custom_target(ubuntu-clock-app_components_QMlFiles ALL SOURCES ${COMPONENTS_QML_JS_FILES})
56
57=== modified file 'app/graphics/CMakeLists.txt'
58--- app/graphics/CMakeLists.txt 2014-09-04 00:00:20 +0000
59+++ app/graphics/CMakeLists.txt 2015-08-11 16:05:23 +0000
60@@ -1,8 +1,17 @@
61-file(GLOB GRAPHICS_FILES *.png *.jpg *.svg)
62+set(GRAPHICS_FILES
63+ Background_Texture.jpg
64+ Hour_Hand@27.png
65+ Inner_Clock_Texture.png
66+ Knob@27.png
67+ Minute_Hand@27.png
68+ Second_Hand@27.png
69+ select-none.svg
70+ select.svg
71+)
72
73 # make the files visible in the qtcreator tree
74 if(NOT "${CMAKE_CURRENT_SOURCE_DIR}" STREQUAL "${CMAKE_CURRENT_BINARY_DIR}")
75- add_custom_target(ubuntu-clock-app_graphics_PNGFiles ALL SOURCES ${IMAGES_PNG_FILES})
76+ add_custom_target(ubuntu-clock-app_graphics_PNGFiles ALL SOURCES ${GRAPHICS_FILES})
77 endif(NOT "${CMAKE_CURRENT_SOURCE_DIR}" STREQUAL "${CMAKE_CURRENT_BINARY_DIR}")
78
79 install(FILES ${GRAPHICS_FILES} DESTINATION ${UBUNTU-CLOCK_APP_DIR}/graphics)
80
81=== modified file 'app/upstreamcomponents/CMakeLists.txt'
82--- app/upstreamcomponents/CMakeLists.txt 2014-08-08 10:08:43 +0000
83+++ app/upstreamcomponents/CMakeLists.txt 2015-08-11 16:05:23 +0000
84@@ -1,4 +1,11 @@
85-file(GLOB UPSTREAM_COMPONENTS_QML_JS_FILES *.qml *.js)
86+set(UPSTREAM_COMPONENTS_QML_JS_FILES
87+ FakeHeader.qml
88+ FastScroll.js
89+ FastScroll.qml
90+ ListItemWithActions.qml
91+ ListItemWithActionsCheckBox.qml
92+ PageWithBottomEdge.qml
93+)
94
95 # make the files visible in the qtcreator tree
96 add_custom_target(ubuntu-clock-app_upstream_components_QMlFiles ALL SOURCES ${UPSTREAM_COMPONENTS_QML_JS_FILES})
97
98=== modified file 'app/worldclock/CMakeLists.txt'
99--- app/worldclock/CMakeLists.txt 2014-08-07 20:47:53 +0000
100+++ app/worldclock/CMakeLists.txt 2015-08-11 16:05:23 +0000
101@@ -1,4 +1,9 @@
102-file(GLOB WORLDCLOCK_QML_JS_FILES *.xml *.qml *.js)
103+set(WORLDCLOCK_QML_JS_FILES
104+ AddWorldCityButton.qml
105+ UserWorldCityDelegate.qml
106+ UserWorldCityList.qml
107+ WorldCityList.qml
108+)
109
110 # make the files visible in the qtcreator tree
111 add_custom_target(ubuntu-clock-app_worldclock_QMlFiles ALL SOURCES ${WORLDCLOCK_QML_JS_FILES})

Subscribers

People subscribed via source and target branches

to all changes: