Merge lp://qastaging/~pete-woods/indicator-sound/gettext into lp://qastaging/indicator-sound/13.10

Proposed by Pete Woods
Status: Merged
Approved by: Charles Kerr
Approved revision: 385
Merged at revision: 386
Proposed branch: lp://qastaging/~pete-woods/indicator-sound/gettext
Merge into: lp://qastaging/indicator-sound/13.10
Diff against target: 12 lines (+1/-1)
1 file modified
CMakeLists.txt (+1/-1)
To merge this branch: bzr merge lp://qastaging/~pete-woods/indicator-sound/gettext
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Charles Kerr (community) Approve
Review via email: mp+188618@code.qastaging.launchpad.net

Commit message

Fix translation support

dh_translations doesn't perform variable substitution when parsing the CMakeLists.txt file

Description of the change

Fix translation support

dh_translations doesn't perform variable substitution when parsing the CMakeLists.txt file

To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) wrote :

Pete, indicator-network, indicator-location, and indicator-session all use "set(GETTEXT_PACKAGE ${CMAKE_PROJECT_NAME})" too.

Is this fix a one-off workaround only for i-sound, or should it be copied to these other packages too?

Revision history for this message
Pete Woods (pete-woods) wrote :

> Pete, indicator-network, indicator-location, and indicator-session all use
> "set(GETTEXT_PACKAGE ${CMAKE_PROJECT_NAME})" too.
>
> Is this fix a one-off workaround only for i-sound, or should it be copied to
> these other packages too?

You're absolutely correct - it should be copied to the others.

Revision history for this message
Charles Kerr (charlesk) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches