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
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2013-10-08 13:00:40 +0000
+++ CMakeLists.txt 2013-10-08 22:43:00 +0000
@@ -44,10 +44,12 @@
4444
45if(NOT CLICK_MODE)45if(NOT CLICK_MODE)
46 set(NOTES_APP_DIR ${CMAKE_INSTALL_DATADIR}/${NOTES_APP})46 set(NOTES_APP_DIR ${CMAKE_INSTALL_DATADIR}/${NOTES_APP})
47 set(DESKTOP_PATH "Path=${CMAKE_INSTALL_FULL_DATADIR}/${NOTES_APP}")
47 set(DESKTOP_EXEC 48 set(DESKTOP_EXEC
48 "qmlscene ${CMAKE_INSTALL_FULL_DATADIR}/${NOTES_APP}/NotesApp.qml")49 "qmlscene ${CMAKE_INSTALL_FULL_DATADIR}/${NOTES_APP}/NotesApp.qml")
49else(NOT CLICK_MODE)50else(NOT CLICK_MODE)
50 set(NOTES_APP_DIR "/")51 set(NOTES_APP_DIR "/")
52 set(DESKTOP_PATH "")
51 set(DESKTOP_EXEC 53 set(DESKTOP_EXEC
52 "qmlscene $@ -I ./usr/lib/${CMAKE_LIBRARY_ARCHITECTURE}/qt5/qml NotesApp.qml")54 "qmlscene $@ -I ./usr/lib/${CMAKE_LIBRARY_ARCHITECTURE}/qt5/qml NotesApp.qml")
53endif(NOT CLICK_MODE)55endif(NOT CLICK_MODE)
@@ -72,6 +74,7 @@
72foreach(LINE ${DESKTOP_FILE_CONTENTS})74foreach(LINE ${DESKTOP_FILE_CONTENTS})
73 string(REGEX REPLACE "tr\\\(\"(.*)\"\\\)" "\\1" LINE "${LINE}")75 string(REGEX REPLACE "tr\\\(\"(.*)\"\\\)" "\\1" LINE "${LINE}")
74 string(REGEX REPLACE "\@DESKTOP_EXEC\@" "${DESKTOP_EXEC}" LINE "${LINE}")76 string(REGEX REPLACE "\@DESKTOP_EXEC\@" "${DESKTOP_EXEC}" LINE "${LINE}")
77 string(REGEX REPLACE "\@DESKTOP_PATH\@" "${DESKTOP_PATH}" LINE "${LINE}")
75 file(APPEND ${CMAKE_CURRENT_BINARY_DIR}/${DESKTOP_FILE} "${LINE}\n")78 file(APPEND ${CMAKE_CURRENT_BINARY_DIR}/${DESKTOP_FILE} "${LINE}\n")
76endforeach(LINE)79endforeach(LINE)
7780
7881
=== modified file 'click/manifest.json.in'
--- click/manifest.json.in 2013-09-23 13:47:21 +0000
+++ click/manifest.json.in 2013-10-08 22:43:00 +0000
@@ -1,7 +1,7 @@
1{1{
2 "architecture": "armhf",2 "architecture": "armhf",
3 "description": 3 "description":
4 "An application to take pictures and videos with the device cameras",4 "Note taking application for the Ubuntu platform",
5 "framework": "ubuntu-sdk-13.10", 5 "framework": "ubuntu-sdk-13.10",
6 "hooks": {6 "hooks": {
7 "notes": {7 "notes": {
@@ -13,7 +13,7 @@
13 "maintainer": "Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>",13 "maintainer": "Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>",
14 "name": "com.ubuntu.notes", 14 "name": "com.ubuntu.notes",
15 "title": "Notes", 15 "title": "Notes",
16 "version": "1.4",16 "version": "1.4.@BZR_REVNO@",
17 "x-source": {17 "x-source": {
18 "vcs-bzr": "lp:notes-app", 18 "vcs-bzr": "lp:notes-app",
19 "vcs-bzr-revno": "@BZR_REVNO@"19 "vcs-bzr-revno": "@BZR_REVNO@"
2020
=== modified file 'click/notes-apparmor.json'
--- click/notes-apparmor.json 2013-09-23 13:47:21 +0000
+++ click/notes-apparmor.json 2013-10-08 22:43:00 +0000
@@ -1,4 +1,4 @@
1{1{
2 "policy_groups": [], 2 "policy_groups": ["networking"],
3 "policy_version": 1.03 "policy_version": 1.0
4}4}
55
=== modified file 'notes-app.desktop.in'
--- notes-app.desktop.in 2013-10-08 09:52:53 +0000
+++ notes-app.desktop.in 2013-10-08 22:43:00 +0000
@@ -1,12 +1,11 @@
1[Desktop Entry]1[Desktop Entry]
2Encoding=UTF-8
3Type=Application2Type=Application
4Exec=@DESKTOP_EXEC@3Exec=@DESKTOP_EXEC@
5Path=/usr/share/notes-app/
6Name=tr("Notes")4Name=tr("Notes")
7GenericName=tr("Notes application for Ubuntu")5GenericName=tr("Notes application for Ubuntu")
8Icon=notepad6Icon=notepad
9Terminal=false7Terminal=false
8@DESKTOP_PATH@
10X-Ubuntu-Touch=true9X-Ubuntu-Touch=true
11X-Ubuntu-StageHint=SideStage10X-Ubuntu-StageHint=SideStage
12X-Ubuntu-Gettext-Domain=com.ubuntu.notes11X-Ubuntu-Gettext-Domain=com.ubuntu.notes
1312
=== modified file 'tests/autopilot/notes_app/tests/__init__.py'
--- tests/autopilot/notes_app/tests/__init__.py 2013-10-08 13:00:40 +0000
+++ tests/autopilot/notes_app/tests/__init__.py 2013-10-08 22:43:00 +0000
@@ -79,7 +79,8 @@
7979
80 def launch_click_installed(self):80 def launch_click_installed(self):
81 self.app = self.launch_click_package(81 self.app = self.launch_click_package(
82 "com.ubuntu.notes")82 "com.ubuntu.notes",
83 emulator_base=UbuntuUIToolkitEmulatorBase)
8384
84 def launch_and_quit_app(self):85 def launch_and_quit_app(self):
85 self.launch_app()86 self.launch_app()

Subscribers

People subscribed via source and target branches