Merge lp:~zeller-benjamin/notes-app/sdkcompat into lp:notes-app

Proposed by Benjamin Zeller
Status: Needs review
Proposed branch: lp:~zeller-benjamin/notes-app/sdkcompat
Merge into: lp:notes-app
Diff against target: 115 lines (+30/-13)
5 files modified
CMakeLists.txt (+7/-2)
click/CMakeLists.txt (+8/-1)
click/manifest.json.in (+9/-10)
debian/rules (+3/-0)
src/NotesApp/Plugins/CMakeLists.txt (+3/-0)
To merge this branch: bzr merge lp:~zeller-benjamin/notes-app/sdkcompat
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Ubuntu Phablet Team Pending
Review via email: mp+233892@code.launchpad.net

Commit message

Make notes app cmake project useable out of the Box with QtCreator
- Enable click mode by default
- Disable click mode in debian rules file
- Add variable UBUNTU_MANIFEST_PATH variable so QtC knows where to look
for the manifest file
- Rename notes-apparmor.json => notes.apparmor (editor support)
- Automatically substitute click architecture in the manifest file

Description of the change

Make notes app cmake project useable out of the Box with QtCreator in devices and emulator,
there is still a problem with running it locally, because the QML plugin is in a nested
directory in the builddir, which is currently not supported.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Unmerged revisions

289. By Benjamin Zeller

Make notes app cmake project useable out of the Box with QtCreator
- Enable click mode by default
- Disable click mode in debian rules file
- Add variable UBUNTU_MANIFEST_PATH variable so QtC knows where to look
for the manifest file
- Rename notes-apparmor.json => notes.apparmor (editor support)
- Automatically substitute click architecture in the manifest file

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2014-06-20 16:52:42 +0000
+++ CMakeLists.txt 2014-09-09 09:16:59 +0000
@@ -27,8 +27,8 @@
27 ENABLE_COVERAGE_REPORT(TARGETS ${SHELL_APP})27 ENABLE_COVERAGE_REPORT(TARGETS ${SHELL_APP})
28endif()28endif()
2929
30option(INSTALL_TESTS "Install the tests on make install" on)30option(INSTALL_TESTS "Install the tests on make install" off)
31option(CLICK_MODE "Installs to a contained location" off)31option(CLICK_MODE "Installs to a contained location" on)
3232
33if(CLICK_MODE)33if(CLICK_MODE)
34 set(CMAKE_INSTALL_PREFIX "/")34 set(CMAKE_INSTALL_PREFIX "/")
@@ -43,6 +43,11 @@
43set(NOTES_APP notes-app)43set(NOTES_APP notes-app)
44set(AUTOPILOT_DIR notes_app)44set(AUTOPILOT_DIR notes_app)
45set(ICON_FILE notepad@30.png)45set(ICON_FILE notepad@30.png)
46set(UBUNTU_MANIFEST_PATH "click/manifest.json.in" CACHE INTERNAL "Relative path to the manifest file")
47
48# make non compiled files visible in qtcreator
49file(GLOB_RECURSE NON_COMPILED_FILES *.qml *.js *.py *.svg *.png *.in *.json *.desktop *.pot *.apparmor qmldir)
50add_custom_target(NON_COMPILED_TARGET ALL SOURCES ${NON_COMPILED_FILES})
4651
47if(NOT CLICK_MODE)52if(NOT CLICK_MODE)
48 set(NOTES_APP_DIR ${CMAKE_INSTALL_DATADIR}/${NOTES_APP})53 set(NOTES_APP_DIR ${CMAKE_INSTALL_DATADIR}/${NOTES_APP})
4954
=== modified file 'click/CMakeLists.txt'
--- click/CMakeLists.txt 2013-09-23 16:29:47 +0000
+++ click/CMakeLists.txt 2014-09-09 09:16:59 +0000
@@ -2,9 +2,16 @@
2 if(NOT BZR_REVNO)2 if(NOT BZR_REVNO)
3 set(BZR_REVNO "trunk")3 set(BZR_REVNO "trunk")
4 endif(NOT BZR_REVNO)4 endif(NOT BZR_REVNO)
5
6 execute_process(
7 COMMAND dpkg-architecture -qDEB_HOST_ARCH
8 OUTPUT_VARIABLE CLICK_ARCH
9 OUTPUT_STRIP_TRAILING_WHITESPACE
10 )
11
5 configure_file(manifest.json.in ${CMAKE_CURRENT_BINARY_DIR}/manifest.json)12 configure_file(manifest.json.in ${CMAKE_CURRENT_BINARY_DIR}/manifest.json)
6 install(FILES ${CMAKE_CURRENT_BINARY_DIR}/manifest.json13 install(FILES ${CMAKE_CURRENT_BINARY_DIR}/manifest.json
7 DESTINATION ${CMAKE_INSTALL_PREFIX})14 DESTINATION ${CMAKE_INSTALL_PREFIX})
8 install(FILES notes-apparmor.json15 install(FILES notes.apparmor
9 DESTINATION ${CMAKE_INSTALL_PREFIX})16 DESTINATION ${CMAKE_INSTALL_PREFIX})
10endif(CLICK_MODE)17endif(CLICK_MODE)
1118
=== modified file 'click/manifest.json.in'
--- click/manifest.json.in 2014-04-10 10:38:52 +0000
+++ click/manifest.json.in 2014-09-09 09:16:59 +0000
@@ -1,24 +1,23 @@
1{1{
2 "architecture": "armhf",2 "architecture": "@CLICK_ARCH@",
3 "description": 3 "description": "Note taking application for the Ubuntu platform",
4 "Note taking application for the Ubuntu platform",4 "framework": "ubuntu-sdk-14.04-dev1",
5 "framework": "ubuntu-sdk-14.04-dev1",
6 "hooks": {5 "hooks": {
7 "notes": {6 "notes": {
8 "apparmor": "notes-apparmor.json", 7 "apparmor": "notes.apparmor",
9 "desktop": "@CMAKE_INSTALL_DATADIR@/applications/@DESKTOP_FILE@"8 "desktop": "@CMAKE_INSTALL_DATADIR@/applications/@DESKTOP_FILE@"
10 }9 }
11 }, 10 },
12 "icon": "notepad",11 "icon": "notepad",
13 "maintainer": "Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>",12 "maintainer": "Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>",
14 "name": "com.ubuntu.notes", 13 "name": "com.ubuntu.notes",
15 "title": "Notes", 14 "title": "Notes",
16 "version": "1.4.@BZR_REVNO@",15 "version": "1.4.@BZR_REVNO@",
17 "x-source": {16 "x-source": {
18 "vcs-bzr": "lp:notes-app", 17 "vcs-bzr": "lp:notes-app",
19 "vcs-bzr-revno": "@BZR_REVNO@"18 "vcs-bzr-revno": "@BZR_REVNO@"
20 },19 },
21 "x-test": {20 "x-test": {
22 "autopilot": "@AUTOPILOT_DIR@"21 "autopilot": "@AUTOPILOT_DIR@"
23 }22 }
24}23}
2524
=== renamed file 'click/notes-apparmor.json' => 'click/notes.apparmor'
=== modified file 'debian/rules'
--- debian/rules 2014-04-10 10:38:52 +0000
+++ debian/rules 2014-09-09 09:16:59 +0000
@@ -13,6 +13,9 @@
13%:13%:
14 dh $@ --parallel --with translations,python314 dh $@ --parallel --with translations,python3
1515
16override_dh_auto_configure:
17 dh_auto_configure -- -DCLICK_MODE=OFF -DINSTALL_TESTS=ON
18
16override_dh_install:19override_dh_install:
17 dh_install --fail-missing20 dh_install --fail-missing
1821
1922
=== modified file 'src/NotesApp/Plugins/CMakeLists.txt'
--- src/NotesApp/Plugins/CMakeLists.txt 2013-05-07 15:45:23 +0000
+++ src/NotesApp/Plugins/CMakeLists.txt 2014-09-09 09:16:59 +0000
@@ -22,6 +22,9 @@
22endfunction(QUERY_QMAKE)22endfunction(QUERY_QMAKE)
23query_qmake(QT_INSTALL_QML QT_IMPORTS_DIR)23query_qmake(QT_INSTALL_QML QT_IMPORTS_DIR)
2424
25#make sure that the qmldir file is in the QML2_IMPORT_PATH when running from QtC
26file(COPY qmldir DESTINATION ${CMAKE_CURRENT_BINARY_DIR})
27
25set(PLUGIN_DIR ${QT_IMPORTS_DIR}/NotesApp/Plugins)28set(PLUGIN_DIR ${QT_IMPORTS_DIR}/NotesApp/Plugins)
26install(TARGETS notes-app-plugin DESTINATION ${PLUGIN_DIR})29install(TARGETS notes-app-plugin DESTINATION ${PLUGIN_DIR})
27install(FILES qmldir DESTINATION ${PLUGIN_DIR})30install(FILES qmldir DESTINATION ${PLUGIN_DIR})

Subscribers

People subscribed via source and target branches