Merge lp:~lains/snap-elementary/fix-appid into lp:snap-elementary

Proposed by Lains
Status: Merged
Approved by: Danielle Foré
Approved revision: 402
Merged at revision: 403
Proposed branch: lp:~lains/snap-elementary/fix-appid
Merge into: lp:snap-elementary
Diff against target: 42 lines (+8/-8)
2 files modified
data/CMakeLists.txt (+7/-7)
src/Application.vala (+1/-1)
To merge this branch: bzr merge lp:~lains/snap-elementary/fix-appid
Reviewer Review Type Date Requested Status
Felipe Escoto (community) Approve
Review via email: mp+304590@code.launchpad.net

Commit message

Snap Photobooth "org.pantheon.snap":

* Fixed filenames to the AppID name.
* Fixed code to have it point to a new AppID.

Description of the change

Snap Photobooth "org.pantheon.snap-elementary":

* Fixed filenames to the AppID name.
* Fixed code to have it point to a new AppID.

To post a comment you must log in.
Revision history for this message
Rico Tzschichholz (ricotz) wrote :

I guess "org.pantheon.snap" would suffice.

Revision history for this message
Lains (lains) wrote :

Well, this app is named throughout the code as snap-elementary, I thought that it would be good to keep that naming in the other places.

Revision history for this message
Danielle Foré (danrabbit) wrote :

I believe this is only to avoid naming conflicts. If we're using RDNN that fixes the naming conflicts. So rico is right. No need to have -elementary here

lp:~lains/snap-elementary/fix-appid updated
402. By Lains <email address hidden>

Fix Snap AppID and rename files to the new AppID, part 2

Revision history for this message
Lains (lains) wrote :

And it's fixed.

Revision history for this message
Felipe Escoto (philip.scott) wrote :

Approved, but on hold until Loki is out

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'data/CMakeLists.txt'
--- data/CMakeLists.txt 2016-07-30 18:16:04 +0000
+++ data/CMakeLists.txt 2016-09-01 20:04:22 +0000
@@ -2,16 +2,16 @@
22
3# We need to create two temporary desktop files in order to have all translations available.3# We need to create two temporary desktop files in order to have all translations available.
4set (APP_NAME "Camera")4set (APP_NAME "Camera")
5configure_file(snap-photobooth.desktop.in.in ${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth.desktop.in)5configure_file(org.pantheon.snap.desktop.in.in ${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap.desktop.in)
6set (APP_NAME "Snap")6set (APP_NAME "Snap")
7configure_file(snap-photobooth.desktop.in.in ${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth-original.desktop.in)7configure_file(org.pantheon.snap.desktop.in.in ${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap-original.desktop.in)
8if (BUILD_FOR_ELEMENTARY)8if (BUILD_FOR_ELEMENTARY)
9 configure_file_translation(${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth.desktop.in ${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth.desktop ${CMAKE_SOURCE_DIR}/po/)9 configure_file_translation(${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap.desktop.in ${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap.desktop ${CMAKE_SOURCE_DIR}/po/)
10else ()10else ()
11 configure_file_translation(${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth-original.desktop.in ${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth.desktop ${CMAKE_SOURCE_DIR}/po/)11 configure_file_translation(${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap-original.desktop.in ${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap.desktop ${CMAKE_SOURCE_DIR}/po/)
12endif ()12endif ()
1313
14configure_file_translation(${CMAKE_CURRENT_SOURCE_DIR}/snap-photobooth.appdata.xml.in ${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth.appdata.xml ${CMAKE_SOURCE_DIR}/po/)14configure_file_translation(${CMAKE_CURRENT_SOURCE_DIR}/org.pantheon.snap.appdata.xml.in ${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap.appdata.xml ${CMAKE_SOURCE_DIR}/po/)
1515
16install (FILES ${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth.appdata.xml DESTINATION share/appdata)16install (FILES ${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap.appdata.xml DESTINATION share/appdata)
17install (FILES ${CMAKE_CURRENT_BINARY_DIR}/snap-photobooth.desktop DESTINATION share/applications)17install (FILES ${CMAKE_CURRENT_BINARY_DIR}/org.pantheon.snap.desktop DESTINATION share/applications)
1818
=== renamed file 'data/snap-photobooth.appdata.xml.in' => 'data/org.pantheon.snap.appdata.xml.in'
=== renamed file 'data/snap-photobooth.desktop.in.in' => 'data/org.pantheon.snap.desktop.in.in'
=== modified file 'src/Application.vala'
--- src/Application.vala 2016-07-30 18:16:04 +0000
+++ src/Application.vala 2016-09-01 20:04:22 +0000
@@ -44,7 +44,7 @@
44 app_years = "2011-2016";44 app_years = "2011-2016";
45 app_icon = "accessories-camera";45 app_icon = "accessories-camera";
46 app_launcher = "snap-photobooth.desktop";46 app_launcher = "snap-photobooth.desktop";
47 application_id = "net.launchpad.snap-elementary";47 application_id = "org.pantheon.snap";
48 main_url = "https://launchpad.net/snap-elementary";48 main_url = "https://launchpad.net/snap-elementary";
49 bug_url = "https://bugs.launchpad.net/snap-elementary";49 bug_url = "https://bugs.launchpad.net/snap-elementary";
50 help_url = "https://answers.launchpad.net/snap-elementary";50 help_url = "https://answers.launchpad.net/snap-elementary";

Subscribers

People subscribed via source and target branches

to all changes: