Merge lp:~leonardolemos/screenshot-tool/fix-1561686 into lp:~elementary-apps/screenshot-tool/trunk

[Leonardo Lemos]

Translate the desktop file

This branch upates the elementary's cmake modules and introduces an extra translation template to handle data files, the desktop file, to be more specific.

[lp:~leonardolemos/screenshot-tool/fix-1561686]

160. By Leonardo Lemos on 2016-04-18

Remove X-GNOME-Gettext-Domain

161. By Leonardo Lemos on 2016-04-18

Update translation template

=== modified file 'cmake/Translations.cmake'
--- cmake/Translations.cmake	2014-12-30 22:08:28 +0000
+++ cmake/Translations.cmake	2016-04-19 01:10:35 +0000
@@ -1,4 +1,5 @@
 # Translations.cmake, CMake macros written for Marlin, feel free to re-use them
+include(CMakeParseArguments)
 
 macro (add_translations_directory NLS_PACKAGE)
     add_custom_target (i18n ALL COMMENT “Building i18n messages.”)
@@ -85,16 +86,44 @@
     endif ()
 endmacro (create_po_file)
 
+macro (configure_file_translation SOURCE RESULT PO_DIR)
+    find_program (INTLTOOL_MERGE_EXECUTABLE intltool-merge)
+    set(EXTRA_PO_DIR ${PO_DIR}/extra/)
+    get_filename_component(EXTRA_PO_DIR ${EXTRA_PO_DIR} ABSOLUTE)
+
+    # Intltool can't create a new directory.
+    get_filename_component(SOURCE_DIRECTORY ${SOURCE} DIRECTORY)
+    file(MAKE_DIRECTORY ${SOURCE_DIRECTORY})
+
+    set (INTLTOOL_FLAG "")
+    if (${SOURCE} MATCHES ".desktop")
+        set (INTLTOOL_FLAG "--desktop-style")
+    elseif (${SOURCE} MATCHES ".gschema")
+        set (INTLTOOL_FLAG "--schemas-style")
+    elseif (${SOURCE} MATCHES ".xml")
+        set (INTLTOOL_FLAG "--xml-style")
+    endif ()
+    execute_process (WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} COMMAND ${INTLTOOL_MERGE_EXECUTABLE} --quiet ${INTLTOOL_FLAG} ${EXTRA_PO_DIR} ${SOURCE} ${RESULT})
+endmacro ()
+
 macro (add_translations_catalog NLS_PACKAGE)
+    cmake_parse_arguments (ARGS "" "" "DESKTOP_FILES;APPDATA_FILES;SCHEMA_FILES" ${ARGN})
     add_custom_target (pot COMMENT “Building translation catalog.”)
     find_program (XGETTEXT_EXECUTABLE xgettext)
+    find_program (INTLTOOL_EXTRACT_EXECUTABLE intltool-extract)
+
+    set(EXTRA_PO_DIR ${CMAKE_CURRENT_SOURCE_DIR}/extra)
 
     set(C_SOURCE "")
     set(VALA_SOURCE "")
     set(GLADE_SOURCE "")
 
     foreach(FILES_INPUT ${ARGN})
-        set(BASE_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/${FILES_INPUT})
+        if((${FILES_INPUT} MATCHES ${CMAKE_SOURCE_DIR}) OR (${FILES_INPUT} MATCHES ${CMAKE_BINARY_DIR}))
+            set(BASE_DIRECTORY ${FILES_INPUT})
+        else ()
+            set(BASE_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/${FILES_INPUT})
+        endif ()
 
         file (GLOB_RECURSE SOURCE_FILES RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}/ ${BASE_DIRECTORY}/*.c)
         foreach(C_FILE ${SOURCE_FILES})
@@ -102,22 +131,30 @@
         endforeach()
 
         file (GLOB_RECURSE SOURCE_FILES RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}/ ${BASE_DIRECTORY}/*.vala)
-        foreach(VALA_C_FILE ${SOURCE_FILES})
-            set(VALA_SOURCE ${VALA_SOURCE} ${VALA_C_FILE})
+        foreach(VALA_FILE ${SOURCE_FILES})
+            set(VALA_SOURCE ${VALA_SOURCE} ${VALA_FILE})
         endforeach()
 
         file (GLOB_RECURSE SOURCE_FILES RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}/ ${BASE_DIRECTORY}/*.ui)
-        foreach(GLADE_C_FILE ${SOURCE_FILES})
-            set(GLADE_SOURCE ${GLADE_SOURCE} ${GLADE_C_FILE})
+        foreach(GLADE_FILE ${SOURCE_FILES})
+            set(GLADE_SOURCE ${GLADE_SOURCE} ${GLADE_FILE})
         endforeach()
     endforeach()
 
+    set (XGETTEXT_C_ARGS --add-comments="/" --keyword="_" --keyword="N_" --keyword="C_:1c,2" --keyword="NC_:1c,2" --keyword="ngettext:1,2" --keyword="Q_:1g")
     set(BASE_XGETTEXT_COMMAND
         ${XGETTEXT_EXECUTABLE} -d ${NLS_PACKAGE}
         -o ${CMAKE_CURRENT_SOURCE_DIR}/${NLS_PACKAGE}.pot
-        --add-comments="/" --keyword="_" --keyword="N_" --keyword="C_:1c,2" --keyword="NC_:1c,2" --keyword="ngettext:1,2" --keyword="Q_:1g" --from-code=UTF-8)
-
-   set(CONTINUE_FLAG "")
+        ${XGETTEXT_C_ARGS} --from-code=UTF-8)
+
+    set(EXTRA_XGETTEXT_COMMAND
+        ${XGETTEXT_EXECUTABLE} -d extra
+        -o ${EXTRA_PO_DIR}/extra.pot --no-location --from-code=UTF-8)
+
+    set (INTLTOOL_EXTRACT_COMMAND
+        ${INTLTOOL_EXTRACT_EXECUTABLE} --local --srcdir=/)
+
+    set(CONTINUE_FLAG "")
 
     IF(NOT "${C_SOURCE}" STREQUAL "")
         add_custom_command(TARGET pot WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} COMMAND ${BASE_XGETTEXT_COMMAND} ${C_SOURCE})
@@ -131,5 +168,36 @@
 
     IF(NOT "${GLADE_SOURCE}" STREQUAL "")
         add_custom_command (TARGET pot WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} COMMAND ${BASE_XGETTEXT_COMMAND} ${CONTINUE_FLAG} -LGlade ${GLADE_SOURCE})
-    ENDIF()  
+    ENDIF()
+
+    # We need to create the directory if one extra content exists.
+    IF((NOT "${ARGS_DESKTOP_FILES}" STREQUAL "") OR (NOT "${ARGS_APPDATA_SOURCE}" STREQUAL "") OR (NOT "${ARGS_SCHEMA_SOURCE}" STREQUAL ""))
+        file(MAKE_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/extra/)
+    ENDIF()
+
+    set(CONTINUE_FLAG "")
+
+    foreach(DESKTOP_SOURCE ${ARGS_DESKTOP_FILES})
+        get_filename_component(DESKTOP_SOURCE ${DESKTOP_SOURCE} ABSOLUTE)
+        add_custom_command(TARGET pot WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR} COMMAND ${INTLTOOL_EXTRACT_COMMAND} --type=gettext/keys ${DESKTOP_SOURCE})
+        get_filename_component(DESKTOP_SOURCE_NAME ${DESKTOP_SOURCE} NAME)
+        add_custom_command(TARGET pot WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} COMMAND ${EXTRA_XGETTEXT_COMMAND} ${CONTINUE_FLAG} ${XGETTEXT_C_ARGS} ${CMAKE_CURRENT_BINARY_DIR}/tmp/${DESKTOP_SOURCE_NAME}.h)
+        set(CONTINUE_FLAG "-j")
+    endforeach()
+
+    foreach(APPDATA_SOURCE ${ARGS_APPDATA_FILES})
+        get_filename_component(APPDATA_SOURCE ${APPDATA_SOURCE} ABSOLUTE)
+        add_custom_command(TARGET pot WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR} COMMAND ${INTLTOOL_EXTRACT_COMMAND} --type=gettext/xml ${APPDATA_SOURCE})
+        get_filename_component(APPDATA_SOURCE_NAME ${APPDATA_SOURCE} NAME)
+        add_custom_command(TARGET pot WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} COMMAND ${EXTRA_XGETTEXT_COMMAND} ${CONTINUE_FLAG} ${XGETTEXT_C_ARGS} ${CMAKE_CURRENT_BINARY_DIR}/tmp/${APPDATA_SOURCE_NAME}.h)
+        set(CONTINUE_FLAG "-j")
+    endforeach()
+
+    foreach(SCHEMA_SOURCE ${ARGS_SCHEMA_FILES})
+        get_filename_component(SCHEMA_SOURCE ${SCHEMA_SOURCE} ABSOLUTE)
+        add_custom_command(TARGET pot WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR} COMMAND ${INTLTOOL_EXTRACT_COMMAND} --type=gettext/schemas ${SCHEMA_SOURCE})
+        get_filename_component(SCHEMA_SOURCE_NAME ${SCHEMA_SOURCE} NAME)
+        add_custom_command(TARGET pot WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} COMMAND ${EXTRA_XGETTEXT_COMMAND} ${CONTINUE_FLAG} ${XGETTEXT_C_ARGS} ${CMAKE_CURRENT_BINARY_DIR}/tmp/${SCHEMA_SOURCE_NAME}.h)
+        set(CONTINUE_FLAG "-j")
+    endforeach()
 endmacro ()
 
=== modified file 'cmake/vala/ValaPrecompile.cmake'
--- cmake/vala/ValaPrecompile.cmake	2014-12-30 22:08:28 +0000
+++ cmake/vala/ValaPrecompile.cmake	2016-04-19 01:10:35 +0000
@@ -1,6 +1,6 @@
 ##
 # Copyright 2009-2010 Jakob Westhoff. All rights reserved.
-# Copyright 2012 elementary.
+# Copyright 2012-2016 elementary LLC.
 # 
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are met:
@@ -28,8 +28,8 @@
 # either expressed or implied, of Jakob Westhoff
 ##
 
-include(ParseArguments)
 find_package(Vala REQUIRED)
+include(CMakeParseArguments)
 
 ##
 # Compile vala files to their c equivalents for further processing. 
@@ -115,7 +115,7 @@
 ##
 
 macro(vala_precompile output target_name)
-    parse_arguments(ARGS "TARGET;PACKAGES;OPTIONS;DIRECTORY;GENERATE_GIR;GENERATE_SYMBOLS;GENERATE_HEADER;GENERATE_VAPI;CUSTOM_VAPIS" "" ${ARGN})
+    cmake_parse_arguments (ARGS "" "GENERATE_GIR;GENERATE_SYMBOLS;GENERATE_HEADER;GENERATE_VAPI;DIRECTORY" "PACKAGES;OPTIONS;CUSTOM_VAPIS" ${ARGN})
 
     if(ARGS_DIRECTORY)
         set(DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/${ARGS_DIRECTORY})
@@ -132,7 +132,7 @@
     set(out_files_display "")
     set(${output} "")
 
-    foreach(src ${ARGS_DEFAULT_ARGS})
+    foreach(src ${ARGS_UNPARSED_ARGUMENTS})
         string(REGEX MATCH "^/" IS_MATCHED ${src})
         if(${IS_MATCHED} MATCHES "/")
             set(src_file_path ${src})
@@ -156,12 +156,14 @@
 
     set(custom_vapi_arguments "")
     if(ARGS_CUSTOM_VAPIS)
+        # Check for relative and absolute paths
         foreach(vapi ${ARGS_CUSTOM_VAPIS})
-            if(${vapi} MATCHES ${CMAKE_SOURCE_DIR} OR ${vapi} MATCHES ${CMAKE_BINARY_DIR})
+            string(REGEX MATCH "^/" IS_MATCHED ${vapi})
+            if(${IS_MATCHED} MATCHES "/")
                 list(APPEND custom_vapi_arguments ${vapi})
-            else (${vapi} MATCHES ${CMAKE_SOURCE_DIR} OR ${vapi} MATCHES ${CMAKE_BINARY_DIR})
+            else()
                 list(APPEND custom_vapi_arguments ${CMAKE_CURRENT_SOURCE_DIR}/${vapi})
-            endif(${vapi} MATCHES ${CMAKE_SOURCE_DIR} OR ${vapi} MATCHES ${CMAKE_BINARY_DIR})
+            endif()
         endforeach(vapi ${ARGS_CUSTOM_VAPIS})
     endif(ARGS_CUSTOM_VAPIS)
 
 
=== modified file 'cmake/vala/ValaVersion.cmake'
--- cmake/vala/ValaVersion.cmake	2014-12-27 15:35:33 +0000
+++ cmake/vala/ValaVersion.cmake	2016-04-19 01:10:35 +0000
@@ -27,8 +27,8 @@
 # either expressed or implied, of Jakob Westhoff
 ##
 
-include(ParseArguments)
 find_package(Vala REQUIRED)
+include(CMakeParseArguments)
 
 ##
 # Ensure a certain valac version is available
@@ -52,7 +52,7 @@
 # If no option is specified the version will be treated as a minimal version.
 ##
 macro(ensure_vala_version version)
-    parse_arguments(ARGS "" "MINIMUM;MAXIMUM;EXACT" ${ARGN})
+    cmake_parse_arguments (ARGS "" "MINIMUM;MAXIMUM;EXACT" "" ${ARGN})
 	set(compare_message "")
 	set(error_message "")
 	if(ARGS_MINIMUM)
 
=== renamed file 'data/screenshot.desktop' => 'data/screenshot.desktop.in'
--- data/screenshot.desktop	2016-03-10 00:47:50 +0000
+++ data/screenshot.desktop.in	2016-04-19 01:10:35 +0000
@@ -1,27 +1,26 @@
 [Desktop Entry]
 Type=Application
-Name=Screenshot
-Comment=Save images of the screen
+_Name=Screenshot
+_Comment=Save images of the screen
 Exec=screenshot
 Icon=accessories-screenshot
 Terminal=false
-X-GNOME-Gettext-Domain=screenshot
 Categories=GTK;GNOME;Utility;
-Keywords=Snapshot;Screenshot;
+_Keywords=Snapshot;Screenshot;
 Actions=ScreenShot;WindowShot;AreaShot;AboutDialog;
 
 [Desktop Action AboutDialog]
-Name=About Screenshot
+_Name=About Screenshot
 Exec=screenshot --about
 
 [Desktop Action ScreenShot]
-Name=Screenshot of the Whole Screen
+_Name=Screenshot of the Whole Screen
 Exec=screenshot --screen
 
 [Desktop Action WindowShot]
-Name=Screenshot of the Current Window
+_Name=Screenshot of the Current Window
 Exec=screenshot --window
 
 [Desktop Action AreaShot]
-Name=Screenshot of a Selected Area
+_Name=Screenshot of a Selected Area
 Exec=screenshot --area
 
=== modified file 'po/CMakeLists.txt'
--- po/CMakeLists.txt	2014-12-27 15:35:33 +0000
+++ po/CMakeLists.txt	2016-04-19 01:10:35 +0000
@@ -2,4 +2,6 @@
 add_translations_directory("screenshot")
 add_translations_catalog("screenshot"
     ../src
+    DESKTOP_FILES
+	../data/screenshot.desktop.in
 )
 
=== added directory 'po/extra'
=== added file 'po/extra/extra.pot'
--- po/extra/extra.pot	1970-01-01 00:00:00 +0000
+++ po/extra/extra.pot	2016-04-19 01:10:35 +0000
@@ -0,0 +1,39 @@
+# SOME DESCRIPTIVE TITLE.
+# Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
+# This file is distributed under the same license as the PACKAGE package.
+# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
+#
+#, fuzzy
+msgid ""
+msgstr ""
+"Project-Id-Version: PACKAGE VERSION\n"
+"Report-Msgid-Bugs-To: \n"
+"POT-Creation-Date: 2016-04-18 19:09-0300\n"
+"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
+"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
+"Language-Team: LANGUAGE <[e-mail]>\n"
+"Language: \n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=CHARSET\n"
+"Content-Transfer-Encoding: 8bit\n"
+
+msgid "Screenshot"
+msgstr ""
+
+msgid "Save images of the screen"
+msgstr ""
+
+msgid "Snapshot;Screenshot;"
+msgstr ""
+
+msgid "About Screenshot"
+msgstr ""
+
+msgid "Screenshot of the Whole Screen"
+msgstr ""
+
+msgid "Screenshot of the Current Window"
+msgstr ""
+
+msgid "Screenshot of a Selected Area"
+msgstr ""
 
=== modified file 'po/screenshot.pot'
--- po/screenshot.pot	2016-03-10 00:47:50 +0000
+++ po/screenshot.pot	2016-04-19 01:10:35 +0000
@@ -8,7 +8,7 @@
 msgstr ""
 "Project-Id-Version: PACKAGE VERSION\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2016-03-09 08:19-0600\n"
+"POT-Creation-Date: 2016-04-18 19:09-0300\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <[e-mail]>\n"
@@ -17,58 +17,6 @@
 "Content-Type: text/plain; charset=UTF-8\n"
 "Content-Transfer-Encoding: 8bit\n"
 
-#: ../src/ScreenshotWindow.vala:99
-msgid "Capture area:"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:103
-msgid "Properties:"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:110
-msgid "Grab the whole screen"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:112
-msgid "Grab the current window"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:114
-msgid "Select area to grab"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:125
-msgid "Grab mouse pointer:"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:132
-msgid "Close after saving:"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:139
-msgid "Delay in seconds:"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:155
-msgid "Take Screenshot"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:161 ../src/Widgets/SaveDialog.vala:129
-msgid "Cancel"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:320 ../src/ScreenshotWindow.vala:398
-msgid "Could not capture screenshot"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:321
-msgid "Image not saved"
-msgstr ""
-
-#: ../src/ScreenshotWindow.vala:399
-msgid "Couldn't find an active window"
-msgstr ""
-
 #: ../src/Widgets/SaveDialog.vala:60
 msgid "Screenshot from "
 msgstr ""
@@ -97,11 +45,15 @@
 msgid "Save"
 msgstr ""
 
-#: ../src/Translations.vala:7 ../src/Screenshot.vala:40
+#: ../src/Widgets/SaveDialog.vala:129 ../src/ScreenshotWindow.vala:180
+msgid "Cancel"
+msgstr ""
+
+#: ../src/Translations.vala:7 ../src/Screenshot.vala:56
 msgid "Screenshot"
 msgstr ""
 
-#: ../src/Translations.vala:8 ../src/Screenshot.vala:56
+#: ../src/Translations.vala:8 ../src/Screenshot.vala:72
 msgid "Save images of your screen or individual windows."
 msgstr ""
 
@@ -112,3 +64,83 @@
 #: ../src/Translations.vala:10
 msgid "About Screenshot"
 msgstr ""
+
+#: ../src/ScreenshotWindow.vala:109
+msgid "Capture area:"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:113
+msgid "Properties:"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:120
+msgid "Grab the whole screen"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:122
+msgid "Grab the current window"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:124
+msgid "Select area to grab"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:135
+msgid "Grab mouse pointer:"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:142
+msgid "Close after saving:"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:149
+msgid "Conceal text:"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:156
+msgid "Delay in seconds:"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:174
+msgid "Take Screenshot"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:354 ../src/ScreenshotWindow.vala:427
+msgid "Could not capture screenshot"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:355
+msgid "Image not saved"
+msgstr ""
+
+#: ../src/ScreenshotWindow.vala:428
+msgid "Couldn't find an active window"
+msgstr ""
+
+#: ../src/Screenshot.vala:40
+msgid "Capture active window"
+msgstr ""
+
+#: ../src/Screenshot.vala:41
+msgid "Capture area"
+msgstr ""
+
+#: ../src/Screenshot.vala:42
+msgid "Capture the whole screen"
+msgstr ""
+
+#: ../src/Screenshot.vala:43
+msgid "Take screenshot after specified delay"
+msgstr ""
+
+#: ../src/Screenshot.vala:43
+msgid "Seconds"
+msgstr ""
+
+#: ../src/Screenshot.vala:44
+msgid "Include the pointer with the screenshot"
+msgstr ""
+
+#: ../src/Screenshot.vala:45
+msgid "Redact system text"
+msgstr ""