RomanNum3ral
/
archinstall
mirror of https://github.com/archlinux/archinstall.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Update hardware.py 

Rewrite some function 

if condition is True then return true else return false, transform in return condition directly 

Also I don't understand why we need a try/except at line 151 and why we not write return condition ??
This commit is contained in:
CYPT71 2021-07-22 11:47:56 +02:00 committed by GitHub
parent 3a40764194
commit bbce3aa26a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
archinstall/lib/hardware.py
View File

@ -69,10 +69,7 @@ def has_amd_cpu() -> bool:
def has_intel_cpu() -> bool:
if subprocess.check_output("lscpu | grep Intel", shell=True).strip().decode():
return True
return False
return subprocess.check_output("lscpu | grep Intel", shell=True).strip().decode()
def has_uefi() -> bool:
return os.path.isdir('/sys/firmware/efi')
@ -106,7 +103,7 @@ def cpu_vendor() -> Optional[str]:
for info in cpu_info:
if info.get('field', None) == "Vendor ID:":
return info.get('data', None)
return None
return
def cpu_model() -> Optional[str]:
@ -116,7 +113,7 @@ def cpu_model() -> Optional[str]:
for info in cpu_info:
if info.get('field', None) == "Model name:":
return info.get('data', None)
return None
return
def sys_vendor() -> Optional[str]:
@ -151,6 +148,8 @@ def virtualization() -> Optional[str]:
def is_vm() -> bool:
return b"none" not in b"".join(SysCommand("systemd-detect-virt")).lower()
"""
try:
# systemd-detect-virt issues a non-zero exit code if it is not on a virtual machine
if b"none" not in b"".join(SysCommand("systemd-detect-virt")).lower():
@ -159,5 +158,7 @@ def is_vm() -> bool:
pass
return False
"""
# TODO: Add more identifiers