Merge lp:~mitya57/ubuntu/trusty/memtest86+/launchpad-translations into lp:ubuntu/trusty/memtest86+

Proposed by Dmitry Shachnev
Status: Work in progress
Proposed branch: lp:~mitya57/ubuntu/trusty/memtest86+/launchpad-translations
Merge into: lp:ubuntu/trusty/memtest86+
Diff against target: 33 lines (+10/-2)
2 files modified
debian/changelog (+8/-0)
debian/rules (+2/-2)
To merge this branch: bzr merge lp:~mitya57/ubuntu/trusty/memtest86+/launchpad-translations
Reviewer Review Type Date Requested Status
Dmitry Shachnev Needs Resubmitting
Sebastien Bacher Needs Fixing
Review via email: mp+212087@code.launchpad.net

Description of the change

This MP aims to make memtest86+ translations be picked up by Launchpad.

See https://answers.launchpad.net/launchpad/+question/244439 for the background.

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Trying to build the source from this branch gives

"# make sure PO files are always up-to-date
debconf-updatepo --skip-pot
No PO directory found, use the --podir flag to tell where it is... exiting"

Is that something that needs to be changed as well?

review: Needs Fixing
Revision history for this message
Dmitry Shachnev (mitya57) wrote :

We probably need to split that folder instead of renaming it. Let's do that next cycle, too late for Trusty to break things.

review: Needs Resubmitting

Unmerged revisions

40. By Dmitry Shachnev

Rename debian/po to debian/translations to make it be picked up
by Launchpad (debian/po is a blacklisted location).
Thanks to William Grant for the suggestion.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2014-03-12 13:27:34 +0000
3+++ debian/changelog 2014-03-21 05:29:25 +0000
4@@ -1,3 +1,11 @@
5+memtest86+ (4.20-1.1ubuntu9) trusty; urgency=medium
6+
7+ * Rename debian/po to debian/translations to make it be picked up
8+ by Launchpad (debian/po is a blacklisted location).
9+ Thanks to William Grant for the suggestion.
10+
11+ -- Dmitry Shachnev <mitya57@ubuntu.com> Thu, 20 Mar 2014 21:10:05 +0400
12+
13 memtest86+ (4.20-1.1ubuntu8) trusty; urgency=medium
14
15 * Adds Chinese (zh_TW, zh_HK, zh_CN) translations. (LP: #1261663)
16
17=== modified file 'debian/rules'
18--- debian/rules 2013-12-03 14:43:44 +0000
19+++ debian/rules 2014-03-21 05:29:25 +0000
20@@ -15,9 +15,9 @@
21
22 $(MAKE) # CC=gcc-2.95
23 ./makeiso.sh
24- ls debian/po --hide *.pot --hide *.in | sed 's/\.po//g' | xargs -I {} -t sh -c \
25+ ls debian/translations --hide *.pot --hide *.in | sed 's/\.po//g' | xargs -I {} -t sh -c \
26 "mkdir -p debian/locale/{}/LC_MESSAGES/ && \
27- msgfmt debian/po/{}.po -o debian/locale/{}/LC_MESSAGES/memtest86+.mo"
28+ msgfmt debian/translations/{}.po -o debian/locale/{}/LC_MESSAGES/memtest86+.mo"
29 touch build-stamp
30
31 clean:
32
33=== renamed directory 'debian/po' => 'debian/translations'

Subscribers

People subscribed via source and target branches