Merge lp:~seb128/ubuntu-system-settings/build-translations-template into lp:ubuntu-system-settings

Proposed by Sebastien Bacher on 2014-11-05
Status: Merged
Approved by: Iain Lane on 2014-11-12
Approved revision: 1194
Merged at revision: 1200
Proposed branch: lp:~seb128/ubuntu-system-settings/build-translations-template
Merge into: lp:ubuntu-system-settings
Diff against target: 14 lines (+4/-0)
1 file modified
debian/rules (+4/-0)
To merge this branch: bzr merge lp:~seb128/ubuntu-system-settings/build-translations-template
Reviewer Review Type Date Requested Status
Iain Lane 2014-11-05 Approve on 2014-11-12
PS Jenkins bot continuous-integration Needs Fixing on 2014-11-10
David Planella (community) Needs Information on 2014-11-05
Review via email: mp+240696@code.launchpad.net

Commit Message

Update the translations template during the build

Description of the Change

Update the translations template during the build

To post a comment you must log in.
David Planella (dpm) wrote :

LGTM!

Somehow related: I've looked at po/CMakeLists.txt while doing the review to see what the 'pot' target does. One thing I've noticed is that there is a static list of languages to be built [1]. Could that not be determined dynamically, so that when there is a new language added, the CMakeLists.txt file does not need to be changed?

set(languages "ar;ast;be;bg;bs;ca;cs;de;el;en_AU;en_GB;eo;es;fa;fi;fr;gl;gu;he;hi;hr;hu;id;it;km;ko;lo;lv;ms;my;nb;pl;pt_BR;pt;ru;shn;sl;sv;tr;ug;uk;xh;zh_CN;zh_HK;zh_TW")

[1] http://bazaar.launchpad.net/~system-settings-touch/ubuntu-system-settings/trunk/view/head:/po/CMakeLists.txt#L78

review: Needs Information
1183. By Albert Astals Cid on 2014-11-05

Use AbstractButton for haptic feedback Fixes: 1389630
Approved by: Sebastien Bacher

1184. By Mathieu Trudel-Lapierre on 2014-11-05

[bluetooth] make the bluetooth agent a notifyable property. Fixes: 1346483
Approved by: Iain Lane, PS Jenkins bot

1185. By Ken VanDine on 2014-11-05

[orientation] String change to Rotation Lock
 Fixes: 1388888
Approved by: Sebastien Bacher

1186. By Sebastien Bacher on 2014-11-05

[notifications] update strings to match the design
 Fixes: 1377286
Approved by: Ken VanDine

1187. By PS Jenkins bot on 2014-11-05

Releasing 0.3+15.04.20141105-0ubuntu1

1188. By Nick Dedekind on 2014-11-05

Fixed switches & check-boxes going out of sync with backends. Fixes: 1336715

1189. By kevin gunn on 2014-11-05

fix white space issues

1190. By PS Jenkins bot on 2014-11-05

Releasing 0.3+15.04.20141105.1-0ubuntu1

1191. By PS Jenkins bot on 2014-11-08

Resync trunk

1192. By Launchpad Translations on behalf of system-settings-touch on 2014-11-08

Launchpad automatic translations update.

1193. By Launchpad Translations on behalf of system-settings-touch on 2014-11-09

Launchpad automatic translations update.

Sebastien Bacher (seb128) wrote :

The coded list of languages is a known issue from the cmake transition, I'm unsure how to fix it and I don't think the template update work should be blocked on that since that's not related to the changes made here

1194. By Sebastien Bacher on 2014-11-10

Update the translations template during the build

Sebastien Bacher (seb128) wrote :

(updated to not use dh_translations, that's not needed)

Iain Lane (laney) wrote :

Okay, seems sensible

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/rules'
2--- debian/rules 2014-08-05 10:25:11 +0000
3+++ debian/rules 2014-11-10 16:39:44 +0000
4@@ -12,6 +12,10 @@
5 # So until Debian bug 719148 is fixed, do it ourselves.
6 dh_auto_configure -- -DCMAKE_INSTALL_LOCALSTATEDIR="/var"
7
8+override_dh_auto_build:
9+ dh_auto_build
10+ $(MAKE) -C obj-$(DEB_HOST_GNU_TYPE) pot
11+
12 override_dh_install:
13 cd tests/autopilot; \
14 set -ex; for python in $(shell py3versions -r); do \

Subscribers

People subscribed via source and target branches