Merge lp:~sergiusens/notes-app/click-fixes into lp:notes-app

Proposed by Sergio Schvezov
Status: Merged
Approved by: Loïc Minier
Approved revision: 210
Merged at revision: 209
Proposed branch: lp:~sergiusens/notes-app/click-fixes
Merge into: lp:notes-app
Diff against target: 88 lines (+9/-6)
5 files modified
CMakeLists.txt (+3/-0)
click/manifest.json.in (+2/-2)
click/notes-apparmor.json (+1/-1)
notes-app.desktop.in (+1/-2)
tests/autopilot/notes_app/tests/__init__.py (+2/-1)
To merge this branch: bzr merge lp:~sergiusens/notes-app/click-fixes
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Jamie Strandboge Approve
Ubuntu Phablet Team Pending
Review via email: mp+189980@code.launchpad.net

Commit message

Description fix, easier version tracking, missing network apparmor policy and adding emulator for tests to run.

Description of the change

Description fix, easier version tracking, missing network apparmor policy and adding emulator for tests to run.

http://paste.ubuntu.com/6211435/

To post a comment you must log in.
lp:~sergiusens/notes-app/click-fixes updated
209. By Sergio Schvezov

Removing encoding

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
lp:~sergiusens/notes-app/click-fixes updated
210. By Sergio Schvezov

Path in desktop conditionalized

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Looks reasonable

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
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 'CMakeLists.txt'
2--- CMakeLists.txt 2013-10-08 13:00:40 +0000
3+++ CMakeLists.txt 2013-10-08 22:43:00 +0000
4@@ -44,10 +44,12 @@
5
6 if(NOT CLICK_MODE)
7 set(NOTES_APP_DIR ${CMAKE_INSTALL_DATADIR}/${NOTES_APP})
8+ set(DESKTOP_PATH "Path=${CMAKE_INSTALL_FULL_DATADIR}/${NOTES_APP}")
9 set(DESKTOP_EXEC
10 "qmlscene ${CMAKE_INSTALL_FULL_DATADIR}/${NOTES_APP}/NotesApp.qml")
11 else(NOT CLICK_MODE)
12 set(NOTES_APP_DIR "/")
13+ set(DESKTOP_PATH "")
14 set(DESKTOP_EXEC
15 "qmlscene $@ -I ./usr/lib/${CMAKE_LIBRARY_ARCHITECTURE}/qt5/qml NotesApp.qml")
16 endif(NOT CLICK_MODE)
17@@ -72,6 +74,7 @@
18 foreach(LINE ${DESKTOP_FILE_CONTENTS})
19 string(REGEX REPLACE "tr\\\(\"(.*)\"\\\)" "\\1" LINE "${LINE}")
20 string(REGEX REPLACE "\@DESKTOP_EXEC\@" "${DESKTOP_EXEC}" LINE "${LINE}")
21+ string(REGEX REPLACE "\@DESKTOP_PATH\@" "${DESKTOP_PATH}" LINE "${LINE}")
22 file(APPEND ${CMAKE_CURRENT_BINARY_DIR}/${DESKTOP_FILE} "${LINE}\n")
23 endforeach(LINE)
24
25
26=== modified file 'click/manifest.json.in'
27--- click/manifest.json.in 2013-09-23 13:47:21 +0000
28+++ click/manifest.json.in 2013-10-08 22:43:00 +0000
29@@ -1,7 +1,7 @@
30 {
31 "architecture": "armhf",
32 "description":
33- "An application to take pictures and videos with the device cameras",
34+ "Note taking application for the Ubuntu platform",
35 "framework": "ubuntu-sdk-13.10",
36 "hooks": {
37 "notes": {
38@@ -13,7 +13,7 @@
39 "maintainer": "Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>",
40 "name": "com.ubuntu.notes",
41 "title": "Notes",
42- "version": "1.4",
43+ "version": "1.4.@BZR_REVNO@",
44 "x-source": {
45 "vcs-bzr": "lp:notes-app",
46 "vcs-bzr-revno": "@BZR_REVNO@"
47
48=== modified file 'click/notes-apparmor.json'
49--- click/notes-apparmor.json 2013-09-23 13:47:21 +0000
50+++ click/notes-apparmor.json 2013-10-08 22:43:00 +0000
51@@ -1,4 +1,4 @@
52 {
53- "policy_groups": [],
54+ "policy_groups": ["networking"],
55 "policy_version": 1.0
56 }
57
58=== modified file 'notes-app.desktop.in'
59--- notes-app.desktop.in 2013-10-08 09:52:53 +0000
60+++ notes-app.desktop.in 2013-10-08 22:43:00 +0000
61@@ -1,12 +1,11 @@
62 [Desktop Entry]
63-Encoding=UTF-8
64 Type=Application
65 Exec=@DESKTOP_EXEC@
66-Path=/usr/share/notes-app/
67 Name=tr("Notes")
68 GenericName=tr("Notes application for Ubuntu")
69 Icon=notepad
70 Terminal=false
71+@DESKTOP_PATH@
72 X-Ubuntu-Touch=true
73 X-Ubuntu-StageHint=SideStage
74 X-Ubuntu-Gettext-Domain=com.ubuntu.notes
75
76=== modified file 'tests/autopilot/notes_app/tests/__init__.py'
77--- tests/autopilot/notes_app/tests/__init__.py 2013-10-08 13:00:40 +0000
78+++ tests/autopilot/notes_app/tests/__init__.py 2013-10-08 22:43:00 +0000
79@@ -79,7 +79,8 @@
80
81 def launch_click_installed(self):
82 self.app = self.launch_click_package(
83- "com.ubuntu.notes")
84+ "com.ubuntu.notes",
85+ emulator_base=UbuntuUIToolkitEmulatorBase)
86
87 def launch_and_quit_app(self):
88 self.launch_app()

Subscribers

People subscribed via source and target branches