Merge lp:~smoser/curtin/trunk.1539720 into lp:~curtin-dev/curtin/trunk

Proposed by Scott Moser
Status: Merged
Merged at revision: 347
Proposed branch: lp:~smoser/curtin/trunk.1539720
Merge into: lp:~curtin-dev/curtin/trunk
Diff against target: 55 lines (+7/-5)
4 files modified
curtin/block/mkfs.py (+4/-4)
curtin/deps/__init__.py (+1/-0)
debian/control (+1/-0)
tests/unittests/test_block_mkfs.py (+1/-1)
To merge this branch: bzr merge lp:~smoser/curtin/trunk.1539720
Reviewer Review Type Date Requested Status
Ryan Harper (community) Approve
Server Team CI bot continuous-integration Approve
Review via email: mp+284494@code.launchpad.net

Commit message

change mkfs.fat to mkfs.vfat and add dependency.

older versions of dosfstools did not provide a 'mkfs.fat'.
So we just use mkfs.vfat that does.

Description of the change

change mkfs.fat to mkfs.vfat and add dependency.

older versions of dosfstools did not provide a 'mkfs.fat'.
So we just use mkfs.vfat that does.

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)
Revision history for this message
Ryan Harper (raharper) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'curtin/block/mkfs.py'
--- curtin/block/mkfs.py 2016-01-19 19:32:21 +0000
+++ curtin/block/mkfs.py 2016-01-29 18:36:35 +0000
@@ -30,10 +30,10 @@
30 "ext2": "mkfs.ext2",30 "ext2": "mkfs.ext2",
31 "ext3": "mkfs.ext3",31 "ext3": "mkfs.ext3",
32 "ext4": "mkfs.ext4",32 "ext4": "mkfs.ext4",
33 "fat": "mkfs.fat",33 "fat": "mkfs.vfat",
34 "fat12": "mkfs.fat",34 "fat12": "mkfs.vfat",
35 "fat16": "mkfs.fat",35 "fat16": "mkfs.vfat",
36 "fat32": "mkfs.fat",36 "fat32": "mkfs.vfat",
37 "jfs": "jfs_mkfs",37 "jfs": "jfs_mkfs",
38 "ntfs": "mkntfs",38 "ntfs": "mkntfs",
39 "reiserfs": "mkfs.reiserfs",39 "reiserfs": "mkfs.reiserfs",
4040
=== modified file 'curtin/deps/__init__.py'
--- curtin/deps/__init__.py 2016-01-06 17:52:19 +0000
+++ curtin/deps/__init__.py 2016-01-29 18:36:35 +0000
@@ -30,6 +30,7 @@
30 ('file', 'file'),30 ('file', 'file'),
31 ('lvcreate', 'lvm2'),31 ('lvcreate', 'lvm2'),
32 ('mdadm', 'mdadm'),32 ('mdadm', 'mdadm'),
33 ('mkfs.vfat', 'dosfstools'),
33 ('mkfs.btrfs', 'btrfs-tools'),34 ('mkfs.btrfs', 'btrfs-tools'),
34 ('mkfs.ext4', 'e2fsprogs'),35 ('mkfs.ext4', 'e2fsprogs'),
35 ('mkfs.xfs', 'xfsprogs'),36 ('mkfs.xfs', 'xfsprogs'),
3637
=== modified file 'debian/control'
--- debian/control 2015-11-20 20:42:00 +0000
+++ debian/control 2016-01-29 18:36:35 +0000
@@ -26,6 +26,7 @@
26Architecture: all26Architecture: all
27Priority: extra27Priority: extra
28Depends: btrfs-tools,28Depends: btrfs-tools,
29 dosfstools,
29 file,30 file,
30 gdisk,31 gdisk,
31 lvm2,32 lvm2,
3233
=== modified file 'tests/unittests/test_block_mkfs.py'
--- tests/unittests/test_block_mkfs.py 2016-01-19 19:34:01 +0000
+++ tests/unittests/test_block_mkfs.py 2016-01-29 18:36:35 +0000
@@ -79,7 +79,7 @@
79 def test_mkfs_fat(self):79 def test_mkfs_fat(self):
80 conf = self._get_config("fat32")80 conf = self._get_config("fat32")
81 expected_flags = [["-n", "format1"], ["-F", "32"]]81 expected_flags = [["-n", "format1"], ["-F", "32"]]
82 self._run_mkfs_with_config(conf, "mkfs.fat", expected_flags)82 self._run_mkfs_with_config(conf, "mkfs.vfat", expected_flags)
8383
84 def test_mkfs_invalid_fstype(self):84 def test_mkfs_invalid_fstype(self):
85 """Do not proceed if fstype is None or invalid"""85 """Do not proceed if fstype is None or invalid"""

Subscribers

People subscribed via source and target branches