Merge lp:~frankban/lpsetup/bug-1012166-unsupported-lang into lp:lpsetup

Proposed by Francesco Banconi
Status: Merged
Merged at revision: 29
Proposed branch: lp:~frankban/lpsetup/bug-1012166-unsupported-lang
Merge into: lp:lpsetup
Diff against target: 16 lines (+4/-1)
1 file modified
lpsetup/templates/lp-setup-lxc-test (+4/-1)
To merge this branch: bzr merge lp:~frankban/lpsetup/bug-1012166-unsupported-lang
Reviewer Review Type Date Requested Status
Graham Binns (community) code Approve
Review via email: mp+109872@code.launchpad.net

Description of the change

== Changes ==

Added 'env -u LANG' to avoid bzr errors. See the bug description.

To post a comment you must log in.
Revision history for this message
Graham Binns (gmb) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lpsetup/templates/lp-setup-lxc-test'
2--- lpsetup/templates/lp-setup-lxc-test 2012-04-10 16:36:15 +0000
3+++ lpsetup/templates/lp-setup-lxc-test 2012-06-12 15:27:34 +0000
4@@ -13,8 +13,11 @@
5 # touch a; touch b; ssh localhost -- ls "a b"
6 # succeeds, when it should say that the file "a b" does not exist.
7
8+# We remove the LANG env var in the ssh call to avoid bzr errors occurring
9+# when the host sends a LANG not supported by the container.
10+
11 set -uex
12 lxc-start-ephemeral -u {user} -S '{ssh_key_path}' -o {lxc_name} -- \
13- "xvfb-run --error-file=/var/tmp/xvfb-errors.log \
14+ "env -u LANG xvfb-run --error-file=/var/tmp/xvfb-errors.log \
15 --server-args='-screen 0 1024x768x24' \
16 -a $PWD/bin/test --shuffle --subunit $@"

Subscribers

People subscribed via source and target branches

to all changes: