Merge lp:~nobuto/ubuntu/raring/ecryptfs-utils/record-passphrase-dialogue-translatable into lp:ubuntu/raring/ecryptfs-utils

Proposed by Nobuto Murata on 2012-11-12
Status: Merged
Merge reported by: Marc Deslauriers
Merged at revision: not available
Proposed branch: lp:~nobuto/ubuntu/raring/ecryptfs-utils/record-passphrase-dialogue-translatable
Merge into: lp:ubuntu/raring/ecryptfs-utils
Diff against target: 138 lines (+72/-4)
7 files modified
.pc/applied-patches (+1/-0)
.pc/record-passphrase-dialogue-translatable.patch/src/desktop/ecryptfs-record-passphrase (+19/-0)
debian/changelog (+10/-0)
debian/patches/record-passphrase-dialogue-translatable.patch (+35/-0)
debian/patches/series (+1/-0)
debian/rules (+3/-0)
src/desktop/ecryptfs-record-passphrase (+3/-4)
To merge this branch: bzr merge lp:~nobuto/ubuntu/raring/ecryptfs-utils/record-passphrase-dialogue-translatable
Reviewer Review Type Date Requested Status
Marc Deslauriers Approve on 2012-12-07
Ubuntu branches 2012-11-12 Pending
Review via email: mp+133860@code.launchpad.net

Description of the Change

Adding translation markers to the messages shown in the dialogue.

The markers will be removed by override_dh_install to work as an update-notifier hook.
The specification of translatable update-notifier hooks is:
 https://wiki.ubuntu.com/InteractiveUpgradeHooks

Besides removing line breaks(.) to workaround lp bug 1075304. Because if a hook contains line breaks(.) on the description, translations are not loaded.

To post a comment you must log in.
Marc Deslauriers (mdeslaur) wrote :

Thanks for the merge request. The changes look good, although I would really like to see bug 1075304 fixed instead of removing the line breaks, but I guess that isn't a show-stopper.

Could you please include an appropriate debian/changelog entry?

review: Needs Fixing
91. By Nobuto Murata on 2012-12-06

write up changelog

Nobuto Murata (nobuto) wrote :

I have written the changelog. Thanks!

Marc Deslauriers (mdeslaur) wrote :

Looks good, ACK. I've uploaded it to raring. Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file '.pc/applied-patches'
2--- .pc/applied-patches 1970-01-01 00:00:00 +0000
3+++ .pc/applied-patches 2012-12-06 14:48:21 +0000
4@@ -0,0 +1,1 @@
5+record-passphrase-dialogue-translatable.patch
6
7=== removed file '.pc/applied-patches'
8=== added directory '.pc/record-passphrase-dialogue-translatable.patch'
9=== added directory '.pc/record-passphrase-dialogue-translatable.patch/src'
10=== added directory '.pc/record-passphrase-dialogue-translatable.patch/src/desktop'
11=== added file '.pc/record-passphrase-dialogue-translatable.patch/src/desktop/ecryptfs-record-passphrase'
12--- .pc/record-passphrase-dialogue-translatable.patch/src/desktop/ecryptfs-record-passphrase 1970-01-01 00:00:00 +0000
13+++ .pc/record-passphrase-dialogue-translatable.patch/src/desktop/ecryptfs-record-passphrase 2012-12-06 14:48:21 +0000
14@@ -0,0 +1,19 @@
15+Name: Record your encryption passphrase
16+Priority: High
17+OnlyAdminUsers: False
18+DisplayIf: test -e $HOME/.ecryptfs/wrapped-passphrase -a ! -e $HOME/.ecryptfs/.wrapped-passphrase.recorded
19+Terminal: True
20+GettextDomain: ecryptfs-utils
21+Command: "sh -c 'ecryptfs-unwrap-passphrase $HOME/.ecryptfs/wrapped-passphrase 2>/dev/null && echo [Enter] && head -n1 && touch $HOME/.ecryptfs/.wrapped-passphrase.recorded '"
22+Description: To encrypt your home directory or "Private" folder, a strong
23+ passphrase has been automatically generated. Usually your directory is unlocked
24+ with your user password, but if you ever need to manually recover this
25+ directory, you will need this passphrase. Please print or write it down and
26+ store it in a safe location.
27+ .
28+ If you click "Run this action now", enter your login password at the
29+ "Passphrase" prompt and you can display your randomly generated passphrase.
30+ .
31+ Otherwise, you will need to run "ecryptfs-unwrap-passphrase" from the command
32+ line to retrieve and record your generated passphrase.
33+
34
35=== modified file 'debian/changelog'
36--- debian/changelog 2012-10-25 16:13:28 +0000
37+++ debian/changelog 2012-12-06 14:48:21 +0000
38@@ -1,3 +1,13 @@
39+ecryptfs-utils (101-0ubuntu2) raring; urgency=low
40+
41+ * debian/patches/record-passphrase-dialogue-translatable.patch:
42+ - make "Record your encryption passphrase" dialogue translatable
43+ (LP: #982924)
44+ - to workaround lp bug 1075304, removing line breaks(.) in the
45+ dialogue
46+
47+ -- Nobuto MURATA <nobuto@ubuntu.com> Thu, 06 Dec 2012 23:37:38 +0900
48+
49 ecryptfs-utils (101-0ubuntu1) raring; urgency=low
50
51 [ Eric Lammerts ]
52
53=== removed file 'debian/patches/00list'
54=== added file 'debian/patches/record-passphrase-dialogue-translatable.patch'
55--- debian/patches/record-passphrase-dialogue-translatable.patch 1970-01-01 00:00:00 +0000
56+++ debian/patches/record-passphrase-dialogue-translatable.patch 2012-12-06 14:48:21 +0000
57@@ -0,0 +1,35 @@
58+## Description: translatable "Record your encryption passphrase" dialogue
59+## Adding translation markers to the messages shown in the dialogue.
60+## The markers will be removed by override_dh_install to work as an
61+## update-notifier hook.
62+## The specification of translatable update-notifier hooks is:
63+## https://wiki.ubuntu.com/InteractiveUpgradeHooks
64+## Besides removing line breaks(.) to workaround lp bug 1075304.
65+## Author: Nobuto MURATA <nobuto@ubuntu.com>
66+## Bug-Ubuntu: https://launchpad.net/bugs/982924
67+## Forwarded: no
68+--- a/src/desktop/ecryptfs-record-passphrase
69++++ b/src/desktop/ecryptfs-record-passphrase
70+@@ -1,19 +1,18 @@
71+-Name: Record your encryption passphrase
72++_Name: Record your encryption passphrase
73+ Priority: High
74+ OnlyAdminUsers: False
75+ DisplayIf: test -e $HOME/.ecryptfs/wrapped-passphrase -a ! -e $HOME/.ecryptfs/.wrapped-passphrase.recorded
76+ Terminal: True
77+ GettextDomain: ecryptfs-utils
78+ Command: "sh -c 'ecryptfs-unwrap-passphrase $HOME/.ecryptfs/wrapped-passphrase 2>/dev/null && echo [Enter] && head -n1 && touch $HOME/.ecryptfs/.wrapped-passphrase.recorded '"
79+-Description: To encrypt your home directory or "Private" folder, a strong
80++_Description:
81++ To encrypt your home directory or "Private" folder, a strong
82+ passphrase has been automatically generated. Usually your directory is unlocked
83+ with your user password, but if you ever need to manually recover this
84+ directory, you will need this passphrase. Please print or write it down and
85+ store it in a safe location.
86+- .
87+ If you click "Run this action now", enter your login password at the
88+ "Passphrase" prompt and you can display your randomly generated passphrase.
89+- .
90+ Otherwise, you will need to run "ecryptfs-unwrap-passphrase" from the command
91+ line to retrieve and record your generated passphrase.
92+
93
94=== added file 'debian/patches/series'
95--- debian/patches/series 1970-01-01 00:00:00 +0000
96+++ debian/patches/series 2012-12-06 14:48:21 +0000
97@@ -0,0 +1,1 @@
98+record-passphrase-dialogue-translatable.patch
99
100=== modified file 'debian/rules'
101--- debian/rules 2012-10-25 16:13:28 +0000
102+++ debian/rules 2012-12-06 14:48:21 +0000
103@@ -36,6 +36,9 @@
104 chmod 4755 debian/ecryptfs-utils/sbin/mount.ecryptfs_private
105
106 override_dh_install:
107+ # Removing translation markers to work as an update-notifier hook
108+ sed -i 's/^_//' debian/tmp/usr/share/ecryptfs-utils/ecryptfs-record-passphrase
109+
110 dh_install --fail-missing --sourcedir=debian/tmp
111
112 override_dh_python2:
113
114=== modified file 'src/desktop/ecryptfs-record-passphrase'
115--- src/desktop/ecryptfs-record-passphrase 2012-10-25 16:13:28 +0000
116+++ src/desktop/ecryptfs-record-passphrase 2012-12-06 14:48:21 +0000
117@@ -1,19 +1,18 @@
118-Name: Record your encryption passphrase
119+_Name: Record your encryption passphrase
120 Priority: High
121 OnlyAdminUsers: False
122 DisplayIf: test -e $HOME/.ecryptfs/wrapped-passphrase -a ! -e $HOME/.ecryptfs/.wrapped-passphrase.recorded
123 Terminal: True
124 GettextDomain: ecryptfs-utils
125 Command: "sh -c 'ecryptfs-unwrap-passphrase $HOME/.ecryptfs/wrapped-passphrase 2>/dev/null && echo [Enter] && head -n1 && touch $HOME/.ecryptfs/.wrapped-passphrase.recorded '"
126-Description: To encrypt your home directory or "Private" folder, a strong
127+_Description:
128+ To encrypt your home directory or "Private" folder, a strong
129 passphrase has been automatically generated. Usually your directory is unlocked
130 with your user password, but if you ever need to manually recover this
131 directory, you will need this passphrase. Please print or write it down and
132 store it in a safe location.
133- .
134 If you click "Run this action now", enter your login password at the
135 "Passphrase" prompt and you can display your randomly generated passphrase.
136- .
137 Otherwise, you will need to run "ecryptfs-unwrap-passphrase" from the command
138 line to retrieve and record your generated passphrase.
139

Subscribers

People subscribed via source and target branches

to all changes: