Merge lp:~vthompson/ubuntu-weather-app/fix-1478183 into lp:ubuntu-weather-app

Proposed by Victor Thompson on 2015-08-04
Status: Merged
Approved by: Victor Thompson on 2015-08-07
Approved revision: 81
Merged at revision: 82
Proposed branch: lp:~vthompson/ubuntu-weather-app/fix-1478183
Merge into: lp:ubuntu-weather-app
Diff against target: 42 lines (+3/-3)
3 files modified
CMakeLists.txt (+1/-1)
app/CMakeLists.txt (+0/-1)
debian/changelog (+2/-1)
To merge this branch: bzr merge lp:~vthompson/ubuntu-weather-app/fix-1478183
Reviewer Review Type Date Requested Status
Victor Thompson Approve on 2015-08-07
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve on 2015-08-07
Andrew Hayzen 2015-08-04 Approve on 2015-08-06
Review via email: mp+266823@code.launchpad.net

Commit message

* Fix the CMakeLists.txt to include the app icon

Description of the change

* Fix the CMakeLists.txt to include the app icon

To post a comment you must log in.
Andrew Hayzen (ahayzen) wrote :

The icon is back! LGTM :-)

review: Approve
81. By Victor Thompson on 2015-08-07

Merge trunk/reboot and resolve conflicts

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 2015-07-01 12:22:16 +0000
3+++ CMakeLists.txt 2015-08-07 00:12:27 +0000
4@@ -33,7 +33,7 @@
5 set(APP_HARDCODE ubuntu-weather-app)
6 set(MAIN_QML ${APP_HARDCODE}.qml)
7 set(DESKTOP_FILE "${APP_HARDCODE}.desktop")
8-set(ICON_FILE weather-app@30.png)
9+set(ICON weather-app@30.png)
10 set(AUTOPILOT_DIR ubuntu_weather_app)
11
12 # Set install paths
13
14=== modified file 'app/CMakeLists.txt'
15--- app/CMakeLists.txt 2015-07-01 12:38:05 +0000
16+++ app/CMakeLists.txt 2015-08-07 00:12:27 +0000
17@@ -4,7 +4,6 @@
18 endif(NOT "${CMAKE_CURRENT_SOURCE_DIR}" STREQUAL "${CMAKE_CURRENT_BINARY_DIR}")
19
20
21-set(ICON ${ICON_FILE})
22 install(FILES ${ICON} DESTINATION ${CMAKE_INSTALL_PREFIX})
23
24 install(FILES ${MAIN_QML} DESTINATION ${UBUNTU-WEATHER_APP_DIR})
25
26=== modified file 'debian/changelog'
27--- debian/changelog 2015-08-06 01:34:38 +0000
28+++ debian/changelog 2015-08-07 00:12:27 +0000
29@@ -1,4 +1,4 @@
30-ubuntu-weather-app (3.0) UNRELEASED; urgency=medium
31+ubuntu-weather-app (3.0ubuntu1) UNRELEASED; urgency=medium
32
33 [ Victor Thompson ]
34 * Sync debian version with click version.
35@@ -8,6 +8,7 @@
36 * Add conditions text to day delegate.
37 * Default to OWM if the key file for TWC is blank.
38 * Add sunrise and sunset icons from the spec
39+ * Fix the CMakeLists.txt to include the app icon
40
41 [ Andrew Hayzen ]
42 * Add mocked locations for autopilot and add a test using the data

Subscribers

People subscribed via source and target branches

to all changes: