Merge lp:~bfiller/address-book-app/fix-desktop-file into lp:address-book-app

Proposed by Bill Filler
Status: Merged
Approved by: Sergio Schvezov
Approved revision: 4
Merged at revision: 3
Proposed branch: lp:~bfiller/address-book-app/fix-desktop-file
Merge into: lp:address-book-app
Diff against target: 46 lines (+3/-4)
3 files modified
data/address-book-app.desktop.in (+0/-1)
debian/address-book-app.install (+1/-1)
src/artwork/CMakeLists.txt (+2/-2)
To merge this branch: bzr merge lp:~bfiller/address-book-app/fix-desktop-file
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Sergio Schvezov Approve
Review via email: mp+172037@code.launchpad.net

This proposal supersedes a proposal from 2013-06-28.

Commit message

fix desktop file so app will display in lens, use temporary png for icon as svg was not valid

Description of the change

fix desktop file so app will display in lens, use temporary png for icon as svg was not valid

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote : Posted in a previous version of this proposal

looks good

review: Approve
Revision history for this message
Sergio Schvezov (sergiusens) wrote : Posted in a previous version of this proposal

address-book-app.png

also needs to be changed in debian/addres-book-app.install

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:4
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~bfiller/address-book-app/fix-desktop-file/+merge/172037/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/address-book-app-ci/6/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-amd64-ci/6
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-armhf-ci/6
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-armhf-ci/6/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-i386-ci/6

Click here to trigger a rebuild:
http://s-jenkins:8080/job/address-book-app-ci/6/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Sergio Schvezov (sergiusens) wrote :

ok, jenkins built fine now and I added the commit message for it to land

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/address-book-app.desktop.in'
2--- data/address-book-app.desktop.in 2013-06-27 20:05:52 +0000
3+++ data/address-book-app.desktop.in 2013-06-28 14:01:26 +0000
4@@ -2,7 +2,6 @@
5 Encoding=UTF-8
6 Version=1.0
7 Terminal=false
8-NoDisplay=true
9 Type=Application
10 Name=tr("Address Book")
11 GenericName=tr("Address Book")
12
13=== modified file 'debian/address-book-app.install'
14--- debian/address-book-app.install 2013-06-27 23:14:17 +0000
15+++ debian/address-book-app.install 2013-06-28 14:01:26 +0000
16@@ -1,4 +1,4 @@
17 usr/bin/address-book-app
18 usr/share/address-book-app
19 usr/share/applications/address-book-app.desktop
20-usr/share/icons/hicolor/scalable/apps/address-book-app.svg
21+usr/share/icons/hicolor/48x48/apps/address-book-app.png
22
23=== modified file 'src/artwork/CMakeLists.txt'
24--- src/artwork/CMakeLists.txt 2013-06-27 20:05:52 +0000
25+++ src/artwork/CMakeLists.txt 2013-06-28 14:01:26 +0000
26@@ -1,5 +1,5 @@
27 set(ADDRESS_BOOK_APP_ICON
28- address-book-app.svg
29+ address-book-app.png
30 )
31
32 set(ADDRESS_BOOK_APP_ARTWORK
33@@ -11,7 +11,7 @@
34 )
35
36 install(FILES ${ADDRESS_BOOK_APP_ICON}
37- DESTINATION ${CMAKE_INSTALL_DATADIR}/icons/hicolor/scalable/apps
38+ DESTINATION ${CMAKE_INSTALL_DATADIR}/icons/hicolor/48x48/apps
39 )
40
41 install(FILES ${ADDRESS_BOOK_APP_ARTWORK}
42
43=== added file 'src/artwork/address-book-app.png'
44Binary files src/artwork/address-book-app.png 1970-01-01 00:00:00 +0000 and src/artwork/address-book-app.png 2013-06-28 14:01:26 +0000 differ
45=== removed file 'src/artwork/address-book-app.svg'
46Binary files src/artwork/address-book-app.svg 2013-06-27 20:05:52 +0000 and src/artwork/address-book-app.svg 1970-01-01 00:00:00 +0000 differ

Subscribers

People subscribed via source and target branches