Merge lp:~james-w/tarmac/utf8 into lp:~ubuntuone-hackers/tarmac/trunk

Proposed by James Westby
Status: Merged
Approved by: Jonathan Lange
Approved revision: 415
Merged at revision: 417
Proposed branch: lp:~james-w/tarmac/utf8
Merge into: lp:~ubuntuone-hackers/tarmac/trunk
Diff against target: 11 lines (+1/-1)
1 file modified
tarmac/plugins/command.py (+1/-1)
To merge this branch: bzr merge lp:~james-w/tarmac/utf8
Reviewer Review Type Date Requested Status
Jonathan Lange (community) Approve
Review via email: mp+140546@code.launchpad.net

Commit message

Handle unicode output from tests.

Description of the change

Hi,

This is a bugfix we used in CA when we were seeing failures because of some
test output causing decode errors.

Thanks,

James

To post a comment you must log in.
Revision history for this message
Jonathan Lange (jml) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tarmac/plugins/command.py'
--- tarmac/plugins/command.py 2012-03-06 19:55:15 +0000
+++ tarmac/plugins/command.py 2012-12-18 21:19:19 +0000
@@ -185,7 +185,7 @@
185 u'%(output)s') % {185 u'%(output)s') % {
186 'source': self.proposal.source_branch.display_name,186 'source': self.proposal.source_branch.display_name,
187 'target': self.proposal.target_branch.display_name,187 'target': self.proposal.target_branch.display_name,
188 'output': u'\n'.join([stdout_value, stderr_value]),188 'output': u'\n'.join([stdout_value.decode('utf-8', 'ignore'), stderr_value.decode('utf-8', 'ignore')]),
189 }189 }
190 raise VerifyCommandFailed(message, comment)190 raise VerifyCommandFailed(message, comment)
191191

Subscribers

People subscribed via source and target branches