Indicator Location

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:

Merged at revision:

Proposed branch:

Merge into:

Diff against target:

1 file modified

CMakeLists.txt (+1/-1)

To merge this branch:

bzr merge lp:~charlesk/indicator-location/lp-1233679

Undecided

Fix Released

Link a bug report

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

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

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-01:

review: Approve (continuous-integration)

Revision history for this message

Ted Gould (ted) wrote on 2013-10-02:

Begrudgingly approving.

review: Approve

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

People subscribed via source and target branches

to all changes:

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)