napcok/calamares
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Lint.

Browse Source
This commit is contained in:
Teo Mrnjavac 2016-06-17 17:10:09 +02:00
parent e6806048e4
commit 0f8ef4220c
1 changed files with 23 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
src/modules/fstab/main.py
View File

@ -76,8 +76,8 @@ def is_ssd_disk(disk_name):
# Should not happen unless sysfs changes, but better safe than sorry # Should not happen unless sysfs changes, but better safe than sorry
return False return False
with open(filename) as f: with open(filename) as sysfile:
return f.read() == "0\n" return sysfile.read() == "0\n"
def disk_name_for_partition(partition): def disk_name_for_partition(partition):
@ -132,14 +132,14 @@ class FstabGenerator(object):
mkdir_p(os.path.join(self.root_mount_point, "etc")) mkdir_p(os.path.join(self.root_mount_point, "etc"))
crypttab_path = os.path.join(self.root_mount_point, "etc", "crypttab") crypttab_path = os.path.join(self.root_mount_point, "etc", "crypttab")
with open(crypttab_path, "w") as fl: with open(crypttab_path, "w") as crypttab_file:
print(CRYPTTAB_HEADER, file=fl) print(CRYPTTAB_HEADER, file=crypttab_file)
for partition in self.partitions: for partition in self.partitions:
dct = self.generate_crypttab_line_info(partition) dct = self.generate_crypttab_line_info(partition)
if dct: if dct:
self.print_crypttab_line(dct, file=fl) self.print_crypttab_line(dct, file=crypttab_file)
def generate_crypttab_line_info(self, partition): def generate_crypttab_line_info(self, partition):
""" Generates information for each crypttab entry. """ """ Generates information for each crypttab entry. """
@ -172,14 +172,14 @@ class FstabGenerator(object):
mkdir_p(os.path.join(self.root_mount_point, "etc")) mkdir_p(os.path.join(self.root_mount_point, "etc"))
fstab_path = os.path.join(self.root_mount_point, "etc", "fstab") fstab_path = os.path.join(self.root_mount_point, "etc", "fstab")
with open(fstab_path, "w") as fl: with open(fstab_path, "w") as fstab_file:
print(FSTAB_HEADER, file=fl) print(FSTAB_HEADER, file=fstab_file)
for partition in self.partitions: for partition in self.partitions:
dct = self.generate_fstab_line_info(partition) dct = self.generate_fstab_line_info(partition)
if dct: if dct:
self.print_fstab_line(dct, file=fl) self.print_fstab_line(dct, file=fstab_file)
if self.root_is_ssd: if self.root_is_ssd:
# Mount /tmp on a tmpfs # Mount /tmp on a tmpfs
@ -189,22 +189,23 @@ class FstabGenerator(object):
options="defaults,noatime,mode=1777", options="defaults,noatime,mode=1777",
check=0, check=0,
) )
self.print_fstab_line(dct, file=fl) self.print_fstab_line(dct, file=fstab_file)
def generate_fstab_line_info(self, partition): def generate_fstab_line_info(self, partition):
""" Generates information for each fstab entry. """ """ Generates information for each fstab entry. """
fs = partition["fs"] filesystem = partition["fs"]
mount_point = partition["mountPoint"] mount_point = partition["mountPoint"]
disk_name = disk_name_for_partition(partition) disk_name = disk_name_for_partition(partition)
is_ssd = disk_name in self.ssd_disks is_ssd = disk_name in self.ssd_disks
fs = FS_MAP.get(fs, fs) filesystem = FS_MAP.get(filesystem, filesystem)
if not mount_point and not fs == "swap": if not mount_point and not filesystem == "swap":
return None return None
options = self.mount_options.get(fs, self.mount_options["default"]) options = self.mount_options.get(filesystem,
self.mount_options["default"])
if is_ssd: if is_ssd:
extra = self.ssd_extra_mount_options.get(fs) extra = self.ssd_extra_mount_options.get(filesystem)
if extra: if extra:
options += "," + extra options += "," + extra
@ -221,7 +222,7 @@ class FstabGenerator(object):
return dict(device="UUID=" + partition["uuid"], return dict(device="UUID=" + partition["uuid"],
mount_point=mount_point or "swap", mount_point=mount_point or "swap",
fs=fs, fs=filesystem,
options=options, options=options,
check=check, check=check,
) )
@ -248,12 +249,15 @@ def run():
:return: :return:
""" """
gs = libcalamares.globalstorage global_storage = libcalamares.globalstorage
conf = libcalamares.job.configuration conf = libcalamares.job.configuration
partitions = gs.value("partitions") partitions = global_storage.value("partitions")
root_mount_point = gs.value("rootMountPoint") root_mount_point = global_storage.value("rootMountPoint")
mount_options = conf["mountOptions"] mount_options = conf["mountOptions"]
ssd_extra_mount_options = conf.get("ssdExtraMountOptions", {}) ssd_extra_mount_options = conf.get("ssdExtraMountOptions", {})
generator = FstabGenerator(partitions, root_mount_point, mount_options, ssd_extra_mount_options) generator = FstabGenerator(partitions,
root_mount_point,
mount_options,
ssd_extra_mount_options)
return generator.run() return generator.run()