Merge lp:~elementary-apps/pantheon-mail/mail-binary into lp:~elementary-apps/pantheon-mail/trunk

Proposed by Danielle Foré
Status: Merged
Approved by: Danielle Foré
Approved revision: 1969
Merged at revision: 1971
Proposed branch: lp:~elementary-apps/pantheon-mail/mail-binary
Merge into: lp:~elementary-apps/pantheon-mail/trunk
Diff against target: 322 lines (+49/-49)
8 files modified
CMakeLists.txt (+4/-4)
data/CMakeLists.txt (+2/-2)
data/pantheon-mail.appdata.xml (+1/-1)
data/theming/CMakeLists.txt (+1/-1)
data/ui/CMakeLists.txt (+1/-1)
sql/CMakeLists.txt (+1/-1)
src/CMakeLists.txt (+30/-30)
src/client/application/geary-application.vala (+9/-9)
To merge this branch: bzr merge lp:~elementary-apps/pantheon-mail/mail-binary
Reviewer Review Type Date Requested Status
elementary Apps team Pending
Review via email: mp+286235@code.launchpad.net

Commit message

rename 'geary' binary with 'pantheon-mail' and rename conflicting install directories

To post a comment you must log in.
1968. By Danielle Foré

geary binary to pantheon-mail

1969. By Danielle Foré

merge trunk

Revision history for this message
Cody Garver (codygarver) wrote :

One remaining known issue that's blocking this branch from fully address the goal "Don't conflict with Geary":

* icons installed to /usr/share/icons/hicolor/ conflict with geary

Other than that, I approve this branch.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2016-01-28 17:48:53 +0000
+++ CMakeLists.txt 2016-02-17 01:22:00 +0000
@@ -1,4 +1,4 @@
1# Geary build script1# Mail build script
2# Copyright 2011-2015 Yorba Foundation2# Copyright 2011-2015 Yorba Foundation
3#3#
4# Check http://webdev.elementaryos.org/docs/developer-guide/cmake for documentation4# Check http://webdev.elementaryos.org/docs/developer-guide/cmake for documentation
@@ -6,15 +6,15 @@
6cmake_minimum_required(VERSION 2.8)6cmake_minimum_required(VERSION 2.8)
7cmake_policy(VERSION 2.6)7cmake_policy(VERSION 2.6)
88
9project(geary C)9project(pantheon-mail C)
1010
11list(APPEND CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake )11list(APPEND CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake )
1212
13#13#
14# Base bits14# Base bits
15#15#
16set(GETTEXT_PACKAGE "geary")16set(GETTEXT_PACKAGE ${CMAKE_PROJECT_NAME})
17set(EXEC_NAME "geary")17set(EXEC_NAME ${CMAKE_PROJECT_NAME})
18set(APP_NAME "Mail")18set(APP_NAME "Mail")
19set(RELEASE_NAME "Lightweight email client")19set(RELEASE_NAME "Lightweight email client")
20set(DESKTOP_ICON "internet-mail")20set(DESKTOP_ICON "internet-mail")
2121
=== modified file 'data/CMakeLists.txt'
--- data/CMakeLists.txt 2015-11-30 17:25:19 +0000
+++ data/CMakeLists.txt 2016-02-17 01:22:00 +0000
@@ -1,5 +1,5 @@
1#1#
2# Build and install geary.desktop2# Build and install pantheon-mail.desktop
3#3#
44
5configure_file(pantheon-mail.desktop.in ${CMAKE_CURRENT_BINARY_DIR}/pantheon-mail.desktop)5configure_file(pantheon-mail.desktop.in ${CMAKE_CURRENT_BINARY_DIR}/pantheon-mail.desktop)
@@ -13,7 +13,7 @@
13install(FILES mail-attach.contract DESTINATION share/contractor)13install(FILES mail-attach.contract DESTINATION share/contractor)
1414
15# Optional: run update-desktop-database at install time.15# Optional: run update-desktop-database at install time.
16# (This has to happen after the geary.desktop file is installed.)16# (This has to happen after the pantheon-mail.desktop file is installed.)
17if (DESKTOP_UPDATE)17if (DESKTOP_UPDATE)
18 install(18 install(
19 CODE19 CODE
2020
=== modified file 'data/pantheon-mail.appdata.xml'
--- data/pantheon-mail.appdata.xml 2016-02-10 18:30:47 +0000
+++ data/pantheon-mail.appdata.xml 2016-02-17 01:22:00 +0000
@@ -25,7 +25,7 @@
25 </ul>25 </ul>
26 </description>26 </description>
27 <provides>27 <provides>
28 <binary>geary</binary>28 <binary>pantheon-mail</binary>
29 </provides>29 </provides>
30 <developer_name>elementary LLC.</developer_name>30 <developer_name>elementary LLC.</developer_name>
31 <url type="homepage">https://elementary.io/</url>31 <url type="homepage">https://elementary.io/</url>
3232
=== modified file 'data/theming/CMakeLists.txt'
--- data/theming/CMakeLists.txt 2012-06-13 21:23:36 +0000
+++ data/theming/CMakeLists.txt 2016-02-17 01:22:00 +0000
@@ -1,4 +1,4 @@
1set(THEMING_DEST share/geary/theming)1set(THEMING_DEST share/${CMAKE_PROJECT_NAME}/theming)
22
3install(FILES message-viewer.html DESTINATION ${THEMING_DEST})3install(FILES message-viewer.html DESTINATION ${THEMING_DEST})
4install(FILES message-viewer.css DESTINATION ${THEMING_DEST})4install(FILES message-viewer.css DESTINATION ${THEMING_DEST})
55
=== modified file 'data/ui/CMakeLists.txt'
--- data/ui/CMakeLists.txt 2015-12-14 17:47:50 +0000
+++ data/ui/CMakeLists.txt 2016-02-17 01:22:00 +0000
@@ -1,4 +1,4 @@
1set(UI_DEST share/geary/ui)1set(UI_DEST share/${CMAKE_PROJECT_NAME}/ui)
22
3install(FILES accelerators.ui DESTINATION ${UI_DEST})3install(FILES accelerators.ui DESTINATION ${UI_DEST})
4install(FILES account_list.ui DESTINATION ${UI_DEST})4install(FILES account_list.ui DESTINATION ${UI_DEST})
55
=== modified file 'sql/CMakeLists.txt'
--- sql/CMakeLists.txt 2014-12-19 01:00:47 +0000
+++ sql/CMakeLists.txt 2016-02-17 01:22:00 +0000
@@ -1,4 +1,4 @@
1set(SQL_DEST share/geary/sql)1set(SQL_DEST share/${CMAKE_PROJECT_NAME}/sql)
22
3install(FILES version-001.sql DESTINATION ${SQL_DEST})3install(FILES version-001.sql DESTINATION ${SQL_DEST})
4install(FILES version-002.sql DESTINATION ${SQL_DEST})4install(FILES version-002.sql DESTINATION ${SQL_DEST})
55
=== modified file 'src/CMakeLists.txt'
--- src/CMakeLists.txt 2016-02-08 02:26:38 +0000
+++ src/CMakeLists.txt 2016-02-17 01:22:00 +0000
@@ -419,7 +419,7 @@
419# Vapigen419# Vapigen
420find_program(VAPIGEN vapigen)420find_program(VAPIGEN vapigen)
421if (VAPIGEN STREQUAL "VAPIGEN-NOTFOUND")421if (VAPIGEN STREQUAL "VAPIGEN-NOTFOUND")
422 message(FATAL_ERROR "vapigen must be installed to build Geary.")422 message(FATAL_ERROR "vapigen must be installed to build Mail.")
423else ()423else ()
424 message(STATUS "Found vapigen: " ${VAPIGEN})424 message(STATUS "Found vapigen: " ${VAPIGEN})
425endif ()425endif ()
@@ -610,19 +610,19 @@
610610
611# Engine (static library used for building)611# Engine (static library used for building)
612#################################################612#################################################
613vala_precompile(ENGINE_VALA_C geary-static613vala_precompile(ENGINE_VALA_C ${CMAKE_PROJECT_NAME}-static
614 ${ENGINE_SRC}614 ${ENGINE_SRC}
615PACKAGES615PACKAGES
616 ${CONSOLE_PACKAGES}616 ${CONSOLE_PACKAGES}
617 ${ENGINE_PACKAGES}617 ${ENGINE_PACKAGES}
618GENERATE_VAPI618GENERATE_VAPI
619 geary-static619 ${CMAKE_PROJECT_NAME}-static
620OPTIONS620OPTIONS
621 ${VALAC_OPTIONS}621 ${VALAC_OPTIONS}
622)622)
623623
624add_library(geary-static STATIC ${ENGINE_VALA_C})624add_library(${CMAKE_PROJECT_NAME}-static STATIC ${ENGINE_VALA_C})
625target_link_libraries(geary-static ${DEPS_LIBRARIES} sqlite3-unicodesn gthread-2.0)625target_link_libraries(${CMAKE_PROJECT_NAME}-static ${DEPS_LIBRARIES} sqlite3-unicodesn gthread-2.0)
626626
627# WebKitGTK VAPI generation627# WebKitGTK VAPI generation
628#################################################628#################################################
@@ -635,83 +635,83 @@
635 vapigen --pkg=gio-2.0 --pkg=gtk+-3.0 --pkg=libsoup-2.4 --metadatadir=. --metadatadir=${WEBKITGTK_METADATA_DIR} --library=webkitgtk-3.0 --directory=${CMAKE_BINARY_DIR}/src ${WEBKITGTK_CUSTOM_VALA} `pkg-config --variable=girdir gobject-introspection-1.0`/WebKit-3.0.gir635 vapigen --pkg=gio-2.0 --pkg=gtk+-3.0 --pkg=libsoup-2.4 --metadatadir=. --metadatadir=${WEBKITGTK_METADATA_DIR} --library=webkitgtk-3.0 --directory=${CMAKE_BINARY_DIR}/src ${WEBKITGTK_CUSTOM_VALA} `pkg-config --variable=girdir gobject-introspection-1.0`/WebKit-3.0.gir
636)636)
637637
638# Geary client app638# Mail client app
639#################################################639#################################################
640vala_precompile(GEARY_VALA_C geary640vala_precompile(GEARY_VALA_C ${CMAKE_PROJECT_NAME}
641 ${CLIENT_SRC}641 ${CLIENT_SRC}
642PACKAGES642PACKAGES
643 ${CLIENT_PACKAGES}643 ${CLIENT_PACKAGES}
644 ${ENGINE_PACKAGES}644 ${ENGINE_PACKAGES}
645CUSTOM_VAPIS645CUSTOM_VAPIS
646 "${CMAKE_BINARY_DIR}/src/geary-static.vapi"646 "${CMAKE_BINARY_DIR}/src/${CMAKE_PROJECT_NAME}-static.vapi"
647 "${CMAKE_BINARY_DIR}/src/webkitgtk-3.0.vapi"647 "${CMAKE_BINARY_DIR}/src/webkitgtk-3.0.vapi"
648OPTIONS648OPTIONS
649 ${VALAC_OPTIONS}649 ${VALAC_OPTIONS}
650 --vapidir=${CMAKE_BINARY_DIR}/src650 --vapidir=${CMAKE_BINARY_DIR}/src
651)651)
652652
653add_executable(geary ${GEARY_VALA_C})653add_executable(${CMAKE_PROJECT_NAME} ${GEARY_VALA_C})
654target_link_libraries(geary m ${DEPS_LIBRARIES} gthread-2.0 geary-static)654target_link_libraries(${CMAKE_PROJECT_NAME} m ${DEPS_LIBRARIES} gthread-2.0 ${CMAKE_PROJECT_NAME}-static)
655install(TARGETS geary RUNTIME DESTINATION bin)655install(TARGETS ${CMAKE_PROJECT_NAME} RUNTIME DESTINATION bin)
656add_custom_command(656add_custom_command(
657 TARGET657 TARGET
658 geary658 ${CMAKE_PROJECT_NAME}
659 POST_BUILD659 POST_BUILD
660 COMMAND 660 COMMAND
661 ${CMAKE_COMMAND} -E copy geary ${CMAKE_BINARY_DIR}/661 ${CMAKE_COMMAND} -E copy ${CMAKE_PROJECT_NAME} ${CMAKE_BINARY_DIR}/
662)662)
663663
664# GSettings664# GSettings
665# This needs to be here and not in desktop/CMakeLists.txt in order for Geary to run in the build665# This needs to be here and not in desktop/CMakeLists.txt in order for Mail to run in the build
666# directory666# directory
667include(GSettings)667include(GSettings)
668add_schemas(geary ${GSETTINGS_DIR} ${CMAKE_INSTALL_PREFIX})668add_schemas(${CMAKE_PROJECT_NAME} ${GSETTINGS_DIR} ${CMAKE_INSTALL_PREFIX})
669669
670# Console app670# Console app
671#################################################671#################################################
672vala_precompile(CONSOLE_VALA_C geary-console672vala_precompile(CONSOLE_VALA_C ${CMAKE_PROJECT_NAME}-console
673 ${CONSOLE_SRC}673 ${CONSOLE_SRC}
674PACKAGES674PACKAGES
675 ${CONSOLE_PACKAGES}675 ${CONSOLE_PACKAGES}
676 ${ENGINE_PACKAGES}676 ${ENGINE_PACKAGES}
677CUSTOM_VAPIS677CUSTOM_VAPIS
678 "${CMAKE_BINARY_DIR}/src/geary-static.vapi"678 "${CMAKE_BINARY_DIR}/src/${CMAKE_PROJECT_NAME}-static.vapi"
679OPTIONS679OPTIONS
680 ${VALAC_OPTIONS}680 ${VALAC_OPTIONS}
681 --vapidir=${CMAKE_BINARY_DIR}/src681 --vapidir=${CMAKE_BINARY_DIR}/src
682)682)
683683
684add_executable(geary-console ${CONSOLE_VALA_C})684add_executable(${CMAKE_PROJECT_NAME}-console ${CONSOLE_VALA_C})
685target_link_libraries(geary-console ${DEPS_LIBRARIES} gthread-2.0 geary-static)685target_link_libraries(${CMAKE_PROJECT_NAME}-console ${DEPS_LIBRARIES} gthread-2.0 ${CMAKE_PROJECT_NAME}-static)
686add_custom_command(686add_custom_command(
687 TARGET687 TARGET
688 geary-console688 ${CMAKE_PROJECT_NAME}-console
689 POST_BUILD689 POST_BUILD
690 COMMAND 690 COMMAND
691 ${CMAKE_COMMAND} -E copy geary-console ${CMAKE_BINARY_DIR}/691 ${CMAKE_COMMAND} -E copy ${CMAKE_PROJECT_NAME}-console ${CMAKE_BINARY_DIR}/
692)692)
693693
694# Mailer app694# Mailer app
695#################################################695#################################################
696vala_precompile(MAILER_VALA_C geary-mailer696vala_precompile(MAILER_VALA_C ${CMAKE_PROJECT_NAME}-mailer
697 ${MAILER_SRC}697 ${MAILER_SRC}
698PACKAGES698PACKAGES
699 ${ENGINE_PACKAGES}699 ${ENGINE_PACKAGES}
700CUSTOM_VAPIS700CUSTOM_VAPIS
701 "${CMAKE_BINARY_DIR}/src/geary-static.vapi"701 "${CMAKE_BINARY_DIR}/src/${CMAKE_PROJECT_NAME}-static.vapi"
702OPTIONS702OPTIONS
703 ${VALAC_OPTIONS}703 ${VALAC_OPTIONS}
704 --vapidir=${CMAKE_BINARY_DIR}/src704 --vapidir=${CMAKE_BINARY_DIR}/src
705)705)
706706
707add_executable(geary-mailer ${MAILER_VALA_C})707add_executable(${CMAKE_PROJECT_NAME}-mailer ${MAILER_VALA_C})
708target_link_libraries(geary-mailer ${DEPS_LIBRARIES} gthread-2.0 geary-static)708target_link_libraries(${CMAKE_PROJECT_NAME}-mailer ${DEPS_LIBRARIES} gthread-2.0 ${CMAKE_PROJECT_NAME}-static)
709add_custom_command(709add_custom_command(
710 TARGET710 TARGET
711 geary-mailer711 ${CMAKE_PROJECT_NAME}-mailer
712 POST_BUILD712 POST_BUILD
713 COMMAND 713 COMMAND
714 ${CMAKE_COMMAND} -E copy geary-mailer ${CMAKE_BINARY_DIR}/714 ${CMAKE_COMMAND} -E copy ${CMAKE_PROJECT_NAME}-mailer ${CMAKE_BINARY_DIR}/
715)715)
716716
717# Valadoc717# Valadoc
@@ -726,7 +726,7 @@
726 WORKING_DIRECTORY726 WORKING_DIRECTORY
727 ${CMAKE_SOURCE_DIR}/src727 ${CMAKE_SOURCE_DIR}/src
728 COMMAND728 COMMAND
729 ${VALADOC_EXECUTABLE} --force --no-protected --target-glib=${TARGET_GLIB} -b ${CMAKE_CURRENT_SOURCE_DIR} -o ${CMAKE_SOURCE_DIR}/valadoc --package-name=geary --package-version=${VERSION} ${ENGINE_SRC} ${valadoc_pkg_opts} --vapidir=${CMAKE_SOURCE_DIR}/bindings/vapi729 ${VALADOC_EXECUTABLE} --force --no-protected --target-glib=${TARGET_GLIB} -b ${CMAKE_CURRENT_SOURCE_DIR} -o ${CMAKE_SOURCE_DIR}/valadoc --package-name=${CMAKE_PROJECT_NAME} --package-version=${VERSION} ${ENGINE_SRC} ${valadoc_pkg_opts} --vapidir=${CMAKE_SOURCE_DIR}/bindings/vapi
730)730)
731731
732## Make clean: remove copied files732## Make clean: remove copied files
@@ -735,8 +735,8 @@
735 DIRECTORY ..735 DIRECTORY ..
736 APPEND736 APPEND
737 PROPERTY ADDITIONAL_MAKE_CLEAN_FILES737 PROPERTY ADDITIONAL_MAKE_CLEAN_FILES
738 geary738 ${CMAKE_PROJECT_NAME}
739 geary-console739 ${CMAKE_PROJECT_NAME}-console
740 geary-mailer740 geary-mailer
741)741)
742742
743743
=== modified file 'src/client/application/geary-application.vala'
--- src/client/application/geary-application.vala 2016-02-09 18:34:21 +0000
+++ src/client/application/geary-application.vala 2016-02-17 01:22:00 +0000
@@ -13,7 +13,7 @@
1313
14public class GearyApplication : Gtk.Application {14public class GearyApplication : Gtk.Application {
15 public const string NAME = "Mail";15 public const string NAME = "Mail";
16 public const string PRGNAME = "geary";16 public const string PRGNAME = "pantheon-mail";
17 public const string APP_ID = "org.pantheon.mail";17 public const string APP_ID = "org.pantheon.mail";
18 public const string COPYRIGHT = _("2011-2015 Yorba Foundation\n© 2016 elementary LLC.");18 public const string COPYRIGHT = _("2011-2015 Yorba Foundation\n© 2016 elementary LLC.");
19 public const string WEBSITE = "https://elementary.io";19 public const string WEBSITE = "https://elementary.io";
@@ -231,11 +231,11 @@
231 }231 }
232 232
233 public File get_user_data_directory() {233 public File get_user_data_directory() {
234 return File.new_for_path(Environment.get_user_data_dir()).get_child("geary");234 return File.new_for_path(Environment.get_user_data_dir()).get_child("pantheon-mail");
235 }235 }
236 236
237 public File get_user_config_directory() {237 public File get_user_config_directory() {
238 return File.new_for_path(Environment.get_user_config_dir()).get_child("geary");238 return File.new_for_path(Environment.get_user_config_dir()).get_child("pantheon-mail");
239 }239 }
240 240
241 /**241 /**
@@ -246,9 +246,9 @@
246 */246 */
247 public File get_resource_directory() {247 public File get_resource_directory() {
248 if (get_install_dir() != null)248 if (get_install_dir() != null)
249 return get_install_dir().get_child("share").get_child("geary");249 return get_install_dir().get_child("share").get_child("pantheon-mail");
250 else250 else
251 return File.new_for_path(SOURCE_ROOT_DIR);251 return File.new_for_path(SOURCE_ROOT_DIR + "/data");
252 }252 }
253 253
254 // Returns the directory the application is currently executing from.254 // Returns the directory the application is currently executing from.
@@ -259,8 +259,8 @@
259 public File? get_desktop_file() {259 public File? get_desktop_file() {
260 File? install_dir = get_install_dir();260 File? install_dir = get_install_dir();
261 File desktop_file = (install_dir != null)261 File desktop_file = (install_dir != null)
262 ? install_dir.get_child("share").get_child("applications").get_child("geary.desktop")262 ? install_dir.get_child("share").get_child("applications").get_child("pantheon-mail.desktop")
263 : File.new_for_path(SOURCE_ROOT_DIR).get_child("build").get_child("desktop").get_child("geary.desktop");263 : File.new_for_path(SOURCE_ROOT_DIR).get_child("build").get_child("desktop").get_child("pantheon-mail.desktop");
264 264
265 return desktop_file.query_exists() ? desktop_file : null;265 return desktop_file.query_exists() ? desktop_file : null;
266 }266 }
@@ -343,7 +343,7 @@
343 }343 }
344 344
345 // Give asynchronous destroy_async() a chance to complete, but to avoid bug(s) where345 // Give asynchronous destroy_async() a chance to complete, but to avoid bug(s) where
346 // Geary hangs at exit, shut the whole thing down if destroy_async() takes too long to346 // Mail hangs at exit, shut the whole thing down if destroy_async() takes too long to
347 // complete347 // complete
348 int64 start_usec = get_monotonic_time();348 int64 start_usec = get_monotonic_time();
349 destroy_async.begin();349 destroy_async.begin();
@@ -352,7 +352,7 @@
352 352
353 int64 delta_usec = get_monotonic_time() - start_usec;353 int64 delta_usec = get_monotonic_time() - start_usec;
354 if (delta_usec >= FORCE_SHUTDOWN_USEC) {354 if (delta_usec >= FORCE_SHUTDOWN_USEC) {
355 debug("Forcing shutdown of Geary, %ss passed...", (delta_usec / USEC_PER_SEC).to_string());355 debug("Forcing shutdown of Mail, %ss passed...", (delta_usec / USEC_PER_SEC).to_string());
356 356
357 break;357 break;
358 }358 }

Subscribers

People subscribed via source and target branches