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
1=== modified file 'po/CMakeLists.txt'
2--- po/CMakeLists.txt 2013-09-26 21:32:31 +0000
3+++ po/CMakeLists.txt 2013-09-28 23:08:35 +0000
4@@ -10,7 +10,7 @@
5 message(STATUS "gettext found: ${LANGUAGES}")
6 string(REPLACE " " ";" LANGUAGES ${LANGUAGES})
7 foreach(LANG ${LANGUAGES})
8- GETTEXT_PROCESS_PO_FILES(${LANG} ALL)
9+ GETTEXT_PROCESS_PO_FILES(${LANG} ALL PO_FILES ${LANG}.po)
10 install(FILES "${CMAKE_CURRENT_BINARY_DIR}/${LANG}.gmo"
11 DESTINATION "${CMAKE_INSTALL_FULL_LOCALEDIR}/${LANG}/LC_MESSAGES"
12 RENAME "${GETTEXT_PACKAGE}.mo")

Subscribers

People subscribed via source and target branches

to all changes: