Merge lp:~abreu-alexandre/oxide/gettext-domain-oxide-qt into lp:~oxide-developers/oxide/oxide.trunk

Proposed by Alexandre Abreu
Status: Merged
Merged at revision: 754
Proposed branch: lp:~abreu-alexandre/oxide/gettext-domain-oxide-qt
Merge into: lp:~oxide-developers/oxide/oxide.trunk
Diff against target: 59 lines (+8/-5)
3 files modified
CMakeLists.txt (+5/-3)
shared/browser/oxide_browser_context.cc (+1/-1)
shared/shared.gyp (+2/-1)
To merge this branch: bzr merge lp:~abreu-alexandre/oxide/gettext-domain-oxide-qt
Reviewer Review Type Date Requested Status
Chris Coulson Approve
Review via email: mp+234744@code.launchpad.net

Commit message

Make the gexttext domain, "oxide-qt", a proper preprocessor variable instead of a hardcoded constant.

Description of the change

Make the gexttext domain, "oxide-qt", a proper preprocessor variable instead of a hardcoded constant.

To post a comment you must log in.
Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Thanks, feel free to merge this (and in to the 1.2 branch)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2014-09-05 23:19:31 +0000
+++ CMakeLists.txt 2014-09-15 21:03:18 +0000
@@ -65,6 +65,10 @@
65 enable_testing()65 enable_testing()
66endif()66endif()
6767
68# for dh_translations to extract the domain
69# (regarding syntax consistency, see http://pad.lv/1181187)
70set(GETTEXT_PACKAGE "oxide-${OXIDE_BUILD}")
71
68# Import the configuration for this build72# Import the configuration for this build
69include(${CMAKE_SOURCE_DIR}/${OXIDE_BUILD}/config.cmake)73include(${CMAKE_SOURCE_DIR}/${OXIDE_BUILD}/config.cmake)
7074
@@ -129,6 +133,7 @@
129else()133else()
130 list(APPEND _GYP_COMMAND -Denable_chromedriver_build=0)134 list(APPEND _GYP_COMMAND -Denable_chromedriver_build=0)
131endif()135endif()
136list(APPEND GYP_COMMAND -Doxide_gettext_domain=${GETTEXT_PACKAGE})
132137
133if(CMAKE_CROSSCOMPILING)138if(CMAKE_CROSSCOMPILING)
134 if(NOT CHROMIUM_C_HOST_COMPILER OR NOT CHROMIUM_CXX_HOST_COMPILER OR NOT CHROMIUM_HOST_AR)139 if(NOT CHROMIUM_C_HOST_COMPILER OR NOT CHROMIUM_CXX_HOST_COMPILER OR NOT CHROMIUM_HOST_AR)
@@ -176,9 +181,6 @@
176 ${OXIDE_BUILD}/**.cc shared/**.cc)181 ${OXIDE_BUILD}/**.cc shared/**.cc)
177list(SORT I18N_SRC_FILES)182list(SORT I18N_SRC_FILES)
178183
179# for dh_translations to extract the domain
180# (regarding syntax consistency, see http://pad.lv/1181187)
181set(GETTEXT_PACKAGE "oxide-${OXIDE_BUILD}")
182add_subdirectory(po)184add_subdirectory(po)
183185
184add_subdirectory(${OXIDE_BUILD})186add_subdirectory(${OXIDE_BUILD})
185187
=== modified file 'shared/browser/oxide_browser_context.cc'
--- shared/browser/oxide_browser_context.cc 2014-09-11 20:15:31 +0000
+++ shared/browser/oxide_browser_context.cc 2014-09-15 21:03:18 +0000
@@ -213,7 +213,7 @@
213 popup_blocker_enabled(true),213 popup_blocker_enabled(true),
214 host_mapping_rules(params.host_mapping_rules) {214 host_mapping_rules(params.host_mapping_rules) {
215215
216 accept_langs = dgettext("oxide", "AcceptLanguage");216 accept_langs = dgettext(OXIDE_GETTEXT_DOMAIN, "AcceptLanguage");
217 if (accept_langs == "AcceptLanguage") {217 if (accept_langs == "AcceptLanguage") {
218 accept_langs = kDefaultAcceptLanguage;218 accept_langs = kDefaultAcceptLanguage;
219 }219 }
220220
=== modified file 'shared/shared.gyp'
--- shared/shared.gyp 2014-09-02 10:34:20 +0000
+++ shared/shared.gyp 2014-09-15 21:03:18 +0000
@@ -173,7 +173,8 @@
173 '<(DEPTH)/ui/accessibility/accessibility.gyp:accessibility',173 '<(DEPTH)/ui/accessibility/accessibility.gyp:accessibility',
174 ],174 ],
175 'defines': [175 'defines': [
176 'OXIDE_SUBPROCESS_PATH=\"<(oxide_subprocess_path)\"'176 'OXIDE_SUBPROCESS_PATH=\"<(oxide_subprocess_path)\"',
177 'OXIDE_GETTEXT_DOMAIN=\"<(oxide_gettext_domain)\"'
177 ],178 ],
178 'dependencies': [179 'dependencies': [
179 'oxide_packed_resources',180 'oxide_packed_resources',

Subscribers

People subscribed via source and target branches