Merge lp:~zeller-benjamin/ubuntu-weather-app/sdkcompat into lp:ubuntu-weather-app/obsolete.trunk

Proposed by Benjamin Zeller
Status: Merged
Approved by: Alan Pope 🍺🐧🐱 πŸ¦„
Approved revision: 362
Merged at revision: 363
Proposed branch: lp:~zeller-benjamin/ubuntu-weather-app/sdkcompat
Merge into: lp:ubuntu-weather-app/obsolete.trunk
Diff against target: 86 lines (+22/-4)
5 files modified
CMakeLists.txt (+6/-1)
click/CMakeLists.txt (+8/-1)
click/manifest.json.in (+3/-2)
components/CMakeLists.txt (+2/-0)
debian/rules (+3/-0)
To merge this branch: bzr merge lp:~zeller-benjamin/ubuntu-weather-app/sdkcompat
Reviewer Review Type Date Requested Status
Martin Borho Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Alan Pope 🍺🐧🐱 πŸ¦„ Pending
Review via email: mp+233752@code.launchpad.net

Commit message

Make Weather app cmake project useable out of the Box with QtCreator
- Enable click mode by default
- Hook names in the manifest file can not be variables
- Disable click mode in debian rules file
- Add variable UBUNTU_MANIFEST_PATH variable so QtC knows where to look
for the manifest file
- Rename apparmor.json to weather.apparmor (editor support)
- Make sure qml/js files are shown in the project tree

Description of the change

Make Weather app cmake project useable out of the Box with QtCreator
- Enable click mode by default
- Hook names in the manifest file can not be variables
- Disable click mode in debian rules file
- Add variable UBUNTU_MANIFEST_PATH variable so QtC knows where to look
for the manifest file
- Rename apparmor.json to weather.apparmor (editor support)
- Make sure qml/js files are shown in the project tree

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
362. By Benjamin Zeller

Added missing file

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
Martin Borho (martin-borho) wrote :

Looks okay for me, thanks!

review: Approve

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 2014-08-13 07:04:41 +0000
3+++ CMakeLists.txt 2014-09-09 06:12:45 +0000
4@@ -7,7 +7,7 @@
5 include(GNUInstallDirs)
6
7 option(INSTALL_TESTS "Install the tests on make install" on)
8-option(CLICK_MODE "Installs to a contained location" off)
9+option(CLICK_MODE "Installs to a contained location" on)
10
11 set(APP_NAME weather)
12 set(DESKTOP_FILE "${PROJECT_NAME}_${APP_NAME}.desktop")
13@@ -15,6 +15,7 @@
14 set(MAIN_QML ubuntu-weather-app.qml)
15 set(ICON_FILE weather-app@30.png)
16 set(AUTOPILOT_DIR ubuntu_weather_app)
17+set(UBUNTU_MANIFEST_PATH "click/manifest.json.in" CACHE INTERNAL "Relative path to the manifest file")
18
19 if(CLICK_MODE)
20 if(NOT DEFINED BZR_SOURCE)
21@@ -65,6 +66,10 @@
22 # Tests
23 enable_testing()
24
25+file(GLOB QML_JS_FILES *.qml *.js)
26+add_custom_target(ubuntu-weather-app_QMlFiles ALL SOURCES ${QML_JS_FILES})
27+
28 add_subdirectory(click)
29 add_subdirectory(po)
30 add_subdirectory(tests)
31+add_subdirectory(components)
32
33=== modified file 'click/CMakeLists.txt'
34--- click/CMakeLists.txt 2014-01-31 21:06:15 +0000
35+++ click/CMakeLists.txt 2014-09-09 06:12:45 +0000
36@@ -3,6 +3,13 @@
37 set(BZR_REVNO "latest")
38 endif(NOT BZR_REVNO)
39 configure_file(manifest.json.in ${CMAKE_CURRENT_BINARY_DIR}/manifest.json)
40- install(FILES ${CMAKE_CURRENT_BINARY_DIR}/manifest.json apparmor.json
41+ install(FILES ${CMAKE_CURRENT_BINARY_DIR}/manifest.json weather.apparmor
42 DESTINATION ${CMAKE_INSTALL_PREFIX})
43+
44+ # make the click files visible in qtcreator
45+ file(GLOB CLICK_FILES
46+ RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}
47+ *.json *.json.in *.apparmor)
48+
49+ add_custom_target(com_ubuntu_weather_CLICKFiles ALL SOURCES ${CLICK_FILES})
50 endif(CLICK_MODE)
51
52=== modified file 'click/manifest.json.in'
53--- click/manifest.json.in 2014-08-13 07:04:41 +0000
54+++ click/manifest.json.in 2014-09-09 06:12:45 +0000
55@@ -1,9 +1,10 @@
56 {
57+ "architecture": "all",
58 "description": "A weather forecast application for Ubuntu with support for multiple online weather data sources",
59 "framework": "ubuntu-sdk-14.10-qml-dev3",
60 "hooks": {
61- "@APP_NAME@": {
62- "apparmor": "apparmor.json",
63+ "weather": {
64+ "apparmor": "weather.apparmor",
65 "desktop": "@DESKTOP_FILE@"
66 }
67 },
68
69=== renamed file 'click/apparmor.json' => 'click/weather.apparmor'
70=== added file 'components/CMakeLists.txt'
71--- components/CMakeLists.txt 1970-01-01 00:00:00 +0000
72+++ components/CMakeLists.txt 2014-09-09 06:12:45 +0000
73@@ -0,0 +1,2 @@
74+file(GLOB COMPONENTS_QML_JS_FILES *.qml *.js)
75+add_custom_target(ubuntu-weather-app_components_QMlFiles ALL SOURCES ${COMPONENTS_QML_JS_FILES})
76
77=== modified file 'debian/rules'
78--- debian/rules 2014-01-31 21:06:15 +0000
79+++ debian/rules 2014-09-09 06:12:45 +0000
80@@ -6,3 +6,6 @@
81
82 %:
83 dh $@
84+
85+override_dh_auto_configure:
86+ dh_auto_configure -- -DCLICK_MODE=OFF

Subscribers

People subscribed via source and target branches