Midori Web Browser

Merge lp:~midori/midori/pofilesargument into lp:midori

  1. pofilesargument
  2. Merge into trunk
Proposed by Cris Dywan
Status: Merged
Approved by: Paweł Forysiuk
Approved revision: 6420
Merged at revision: 6420
Proposed branch: lp:~midori/midori/pofilesargument
Merge into: lp:midori
Diff against target: 12 lines (+1/-1)
1 file modified
po/CMakeLists.txt (+1/-1)
To merge this branch: bzr merge lp:~midori/midori/pofilesargument
Reviewer Review Type Date Requested Status
Paweł Forysiuk Approve
Review via email: mp+188211@code.launchpad.net

Commit message

Add missing PO_FILES argument to GETTEXT_PROCESS_PO_FILES

Description of the change

Apparently the pofile install code went in with a glaring problem: it doesn't build the mo files reliably and thus fails to install the non-existing files, at least in a clean trunk it doesn't work for me.
This fixes it.

To post a comment you must log in.
Revision history for this message
Paweł Forysiuk (tuxator) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'po/CMakeLists.txt'
--- po/CMakeLists.txt 2013-09-26 21:32:31 +0000
+++ po/CMakeLists.txt 2013-09-28 23:08:35 +0000
@@ -10,7 +10,7 @@
10 message(STATUS "gettext found: ${LANGUAGES}")10 message(STATUS "gettext found: ${LANGUAGES}")
11 string(REPLACE " " ";" LANGUAGES ${LANGUAGES})11 string(REPLACE " " ";" LANGUAGES ${LANGUAGES})
12 foreach(LANG ${LANGUAGES})12 foreach(LANG ${LANGUAGES})
13 GETTEXT_PROCESS_PO_FILES(${LANG} ALL)13 GETTEXT_PROCESS_PO_FILES(${LANG} ALL PO_FILES ${LANG}.po)
14 install(FILES "${CMAKE_CURRENT_BINARY_DIR}/${LANG}.gmo"14 install(FILES "${CMAKE_CURRENT_BINARY_DIR}/${LANG}.gmo"
15 DESTINATION "${CMAKE_INSTALL_FULL_LOCALEDIR}/${LANG}/LC_MESSAGES"15 DESTINATION "${CMAKE_INSTALL_FULL_LOCALEDIR}/${LANG}/LC_MESSAGES"
16 RENAME "${GETTEXT_PACKAGE}.mo")16 RENAME "${GETTEXT_PACKAGE}.mo")

Subscribers

People subscribed via source and target branches

to all changes: