Merge lp:~raharper/curtin/trunk.fix-my-garbage into lp:~curtin-dev/curtin/trunk

Proposed by Scott Moser
Status: Merged
Merged at revision: 452
Proposed branch: lp:~raharper/curtin/trunk.fix-my-garbage
Merge into: lp:~curtin-dev/curtin/trunk
Diff against target: 26 lines (+2/-3)
2 files modified
curtin/block/__init__.py (+0/-1)
tests/vmtests/__init__.py (+2/-2)
To merge this branch: bzr merge lp:~raharper/curtin/trunk.fix-my-garbage
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
curtin developers Pending
Review via email: mp+316654@code.launchpad.net

Commit message

Remove previously committed garbage print and yakkety pep failure

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'curtin/block/__init__.py'
2--- curtin/block/__init__.py 2017-02-06 20:16:35 +0000
3+++ curtin/block/__init__.py 2017-02-07 23:35:38 +0000
4@@ -746,7 +746,6 @@
5 """
6 Obtain an exclusive file-handle to the file/device specified
7 """
8- print('exclusive_open running')
9 mode = 'rb+'
10 fd = None
11 if not os.path.exists(path):
12
13=== modified file 'tests/vmtests/__init__.py'
14--- tests/vmtests/__init__.py 2017-02-07 18:57:35 +0000
15+++ tests/vmtests/__init__.py 2017-02-07 23:35:38 +0000
16@@ -750,8 +750,8 @@
17 return fp.read()
18
19 def load_log_file(self, filename):
20- with open(filename, 'rb') as l:
21- return l.read().decode('utf-8', errors='replace')
22+ with open(filename, 'rb') as fp:
23+ return fp.read().decode('utf-8', errors='replace')
24
25 def get_install_log_curtin_version(self):
26 # curtin: Installation started. (%s)

Subscribers

People subscribed via source and target branches