install: print_flash_info: make non-step (MR 1991)

This doesn't actually do anything else besides printing
advice so it doesn't need to be a processing step.
This commit is contained in:
Johannes Marbach 2020-11-18 19:53:54 +01:00 committed by Oliver Smith
parent c31bf38409
commit ed2175546c
No known key found for this signature in database
GPG key ID: 5AE7F5513E0885CB

View file

@ -514,12 +514,10 @@ def install_system_image(args, size_reserve, suffix, step, steps,
working_dir="/home/pmos/rootfs/") working_dir="/home/pmos/rootfs/")
def print_flash_info(args, step=5, steps=5): def print_flash_info(args):
""" Print flashing information, based on the deviceinfo data and the """ Print flashing information, based on the deviceinfo data and the
pmbootstrap arguments. pmbootstrap arguments. """
logging.info("*** FLASHING INFORMATION ***")
:param step: installation step number """
logging.info(f"*** ({step}/{steps}) FLASHING TO DEVICE ***")
logging.info("Run the following to flash your installation to the" logging.info("Run the following to flash your installation to the"
" target device:") " target device:")
@ -577,14 +575,14 @@ def print_flash_info(args, step=5, steps=5):
" and flash outside of pmbootstrap.") " and flash outside of pmbootstrap.")
def install_recovery_zip(args, step, steps): def install_recovery_zip(args, steps):
logging.info(f"*** ({step}/{steps}) CREATING RECOVERY-FLASHABLE ZIP ***") logging.info(f"*** ({steps}/{steps}) CREATING RECOVERY-FLASHABLE ZIP ***")
suffix = "buildroot_" + args.deviceinfo["arch"] suffix = "buildroot_" + args.deviceinfo["arch"]
mount_device_rootfs(args, f"rootfs_{args.device}", suffix) mount_device_rootfs(args, f"rootfs_{args.device}", suffix)
pmb.install.recovery.create_zip(args, suffix) pmb.install.recovery.create_zip(args, suffix)
# Flash information # Flash information
logging.info(f"*** ({step + 1}/{steps}) FLASHING INFORMATION ***") logging.info("*** FLASHING INFORMATION ***")
logging.info("Flashing with the recovery zip is explained here:") logging.info("Flashing with the recovery zip is explained here:")
logging.info("<https://postmarketos.org/recoveryzip>") logging.info("<https://postmarketos.org/recoveryzip>")
@ -655,11 +653,11 @@ def install(args):
if args.no_image: if args.no_image:
steps = 2 steps = 2
elif args.android_recovery_zip: elif args.android_recovery_zip:
steps = 4 steps = 3
elif args.on_device_installer: elif args.on_device_installer:
steps = 8 steps = 7
else: else:
steps = 5 steps = 4
# Install required programs in native chroot # Install required programs in native chroot
logging.info("*** (1/{}) PREPARE NATIVE CHROOT ***".format(steps)) logging.info("*** (1/{}) PREPARE NATIVE CHROOT ***".format(steps))
@ -721,7 +719,7 @@ def install(args):
if args.no_image: if args.no_image:
return return
elif args.android_recovery_zip: elif args.android_recovery_zip:
return install_recovery_zip(args, steps - 1, steps) return install_recovery_zip(args, steps)
if args.on_device_installer: if args.on_device_installer:
# Runs install_system_image twice # Runs install_system_image twice
@ -729,4 +727,4 @@ def install(args):
else: else:
install_system_image(args, 0, f"rootfs_{args.device}", 3, steps, install_system_image(args, 0, f"rootfs_{args.device}", 3, steps,
split=args.split, sdcard=args.sdcard) split=args.split, sdcard=args.sdcard)
print_flash_info(args, steps, steps) print_flash_info(args)