Merge lp:~charlesk/indicator-location/lp-1233679 into lp:indicator-location/13.10

Proposed by Charles Kerr on 2013-10-01
Status: Merged
Approved by: Ted Gould on 2013-10-02
Approved revision: 48
Merged at revision: 48
Proposed branch: lp:~charlesk/indicator-location/lp-1233679
Merge into: lp:indicator-location/13.10
Diff against target: 12 lines (+1/-1)
1 file modified
CMakeLists.txt (+1/-1)
To merge this branch: bzr merge lp:~charlesk/indicator-location/lp-1233679
Reviewer Review Type Date Requested Status
Ted Gould (community) 2013-10-01 Approve on 2013-10-02
PS Jenkins bot (community) continuous-integration Approve on 2013-10-01
Review via email: mp+188705@code.launchpad.net

Description of the change

One-liner to set GETTEXT_PACKAGE explicitly because dh_translations can't handle ${CMAKE_PROJECT_NAME}

To post a comment you must log in.
Ted Gould (ted) wrote :

Begrudgingly approving.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2013-08-29 17:48:37 +0000
3+++ CMakeLists.txt 2013-10-01 20:03:30 +0000
4@@ -5,7 +5,7 @@
5
6 set (PROJECT_VERSION "13.10.0")
7 set (PACKAGE ${CMAKE_PROJECT_NAME})
8-set (GETTEXT_PACKAGE ${CMAKE_PROJECT_NAME})
9+set (GETTEXT_PACKAGE indicator-location)
10
11 option (enable_tests "Build the package's automatic tests." ON)
12 option (enable_lcov "Generate lcov code coverage reports." ON)

Subscribers

People subscribed via source and target branches