Merge lp:~milo/linaro-image-tools/pep8-fixes into lp:linaro-image-tools/11.11

Proposed by Milo Casagrande
Status: Merged
Approved by: Georgy Redkozubov
Approved revision: no longer in the source branch.
Merged at revision: 626
Proposed branch: lp:~milo/linaro-image-tools/pep8-fixes
Merge into: lp:linaro-image-tools/11.11
Diff against target: 68 lines (+7/-7)
5 files modified
linaro_image_tools/hwpack/tests/test_config.py (+3/-3)
linaro_image_tools/hwpack/tests/test_hardwarepack.py (+1/-1)
linaro_image_tools/media_create/boards.py (+1/-1)
linaro_image_tools/media_create/tests/test_media_create.py (+1/-1)
linaro_image_tools/tests/test_utils.py (+1/-1)
To merge this branch: bzr merge lp:~milo/linaro-image-tools/pep8-fixes
Reviewer Review Type Date Requested Status
Georgy Redkozubov Approve
Review via email: mp+171742@code.launchpad.net

Description of the change

Some PEP8 fixes.

To post a comment you must log in.
Revision history for this message
Georgy Redkozubov (gesha) wrote :

Looks good

review: Approve
626. By Milo Casagrande

PEP8 fixes.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'linaro_image_tools/hwpack/tests/test_config.py'
2--- linaro_image_tools/hwpack/tests/test_config.py 2013-02-18 09:40:15 +0000
3+++ linaro_image_tools/hwpack/tests/test_config.py 2013-06-27 09:08:30 +0000
4@@ -304,9 +304,9 @@
5 def test_validate_partition_layout(self):
6 partition_layout = 'apafs_bananfs'
7 config = self.get_config(self.valid_start_v2 + "u_boot_package = "
8- "u-boot-linaro-s5pv310\nu_boot_file = "
9- "u-boot.bin\npartition_layout = %s\n" %
10- partition_layout)
11+ "u-boot-linaro-s5pv310\nu_boot_file = "
12+ "u-boot.bin\npartition_layout = %s\n" %
13+ partition_layout)
14 self.assertValidationError(
15 "Undefined partition layout %s in the [%s] section. "
16 "Valid partition layouts are %s."
17
18=== modified file 'linaro_image_tools/hwpack/tests/test_hardwarepack.py'
19--- linaro_image_tools/hwpack/tests/test_hardwarepack.py 2013-02-17 13:53:41 +0000
20+++ linaro_image_tools/hwpack/tests/test_hardwarepack.py 2013-06-27 09:08:30 +0000
21@@ -443,7 +443,7 @@
22 self.assertThat(
23 tf,
24 HardwarePackHasFile("pkgs/%s" % package1.filename,
25- content=package1.content.read()))
26+ content=package1.content.read()))
27 self.assertThat(
28 tf,
29 HardwarePackHasFile("pkgs/%s" % package2.filename,
30
31=== modified file 'linaro_image_tools/media_create/boards.py'
32--- linaro_image_tools/media_create/boards.py 2013-05-23 17:29:29 +0000
33+++ linaro_image_tools/media_create/boards.py 2013-06-27 09:08:30 +0000
34@@ -1169,7 +1169,7 @@
35 self.delete_file(toc_filename)
36 if self.delete_startupfiles:
37 self.delete_file(os.path.join(config_files_dir,
38- self.snowball_startup_files_config))
39+ self.snowball_startup_files_config))
40
41 def snowball_config(self, chroot_dir):
42 # We will find the startupfiles in the target boot partition.
43
44=== modified file 'linaro_image_tools/media_create/tests/test_media_create.py'
45--- linaro_image_tools/media_create/tests/test_media_create.py 2013-05-23 17:57:10 +0000
46+++ linaro_image_tools/media_create/tests/test_media_create.py 2013-06-27 09:08:30 +0000
47@@ -2987,7 +2987,7 @@
48 self.linux_offsets_and_sizes = [
49 (16384 * SECTOR_SIZE, 15746 * SECTOR_SIZE),
50 (32768 * SECTOR_SIZE, (self.linux_image_size -
51- 32768 * SECTOR_SIZE))
52+ 32768 * SECTOR_SIZE))
53 ]
54 self.android_image_size = 256 * 1024 ** 2
55 # Extended partition info takes 32 sectors from the first ext partition
56
57=== modified file 'linaro_image_tools/tests/test_utils.py'
58--- linaro_image_tools/tests/test_utils.py 2013-02-25 18:31:26 +0000
59+++ linaro_image_tools/tests/test_utils.py 2013-06-27 09:08:30 +0000
60@@ -280,7 +280,7 @@
61 else:
62 self.assertEqual(
63 ['apt-get -s install dosfstools',
64- '%s apt-get --yes install dosfstools' % sudo_args],
65+ '%s apt-get --yes install dosfstools' % sudo_args],
66 fixture.mock.commands_executed)
67
68 def test_package_installation_refused(self):

Subscribers

People subscribed via source and target branches