Merge lp:~jelmer/brz/create-ssls into lp:brz/3.1

Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~jelmer/brz/create-ssls
Merge into: lp:brz/3.1
Diff against target: 18 lines (+2/-2)
1 file modified
breezy/tests/ssl_certs/create_ssls.py (+2/-2)
To merge this branch: bzr merge lp:~jelmer/brz/create-ssls
Reviewer Review Type Date Requested Status
Bernhard M. Wiedemann (community) Approve
Jelmer Vernooij Approve
Review via email: mp+395224@code.launchpad.net

Commit message

Port create_ssls.py to Python 3.

Description of the change

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve
Revision history for this message
Bernhard M. Wiedemann (ubuntubmw) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'breezy/tests/ssl_certs/create_ssls.py'
--- breezy/tests/ssl_certs/create_ssls.py 2018-11-23 02:12:35 +0000
+++ breezy/tests/ssl_certs/create_ssls.py 2020-12-11 13:02:25 +0000
@@ -1,4 +1,4 @@
1#! /usr/bin/env python1#! /usr/bin/env python3
22
3# Copyright (C) 2007, 2008, 2009, 2017 Canonical Ltd3# Copyright (C) 2007, 2008, 2009, 2017 Canonical Ltd
4#4#
@@ -84,7 +84,7 @@
84 """84 """
85 cmd = ['openssl'] + args85 cmd = ['openssl'] + args
86 proc = Popen(cmd, stdin=PIPE)86 proc = Popen(cmd, stdin=PIPE)
87 (stdout, stderr) = proc.communicate(input)87 (stdout, stderr) = proc.communicate(input.encode('utf-8'))
88 if proc.returncode:88 if proc.returncode:
89 # Basic error handling, all commands should succeed89 # Basic error handling, all commands should succeed
90 raise CalledProcessError(proc.returncode, cmd)90 raise CalledProcessError(proc.returncode, cmd)

Subscribers

People subscribed via source and target branches