From d8af06c936a446b245ae27f3f39baabbf25b946d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mathis=20Dr=C3=B6ge?= Date: Thu, 30 Jun 2022 10:38:37 +0200 Subject: [PATCH] [cli] Rename "yes" -> "skip_prereqs" in _handle_postinstall The name "yes" was a little confusing, you'd think it means "Yes, install all prerequisites" while it actually skips them --- legendary/cli.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/legendary/cli.py b/legendary/cli.py index ad796a1..039c509 100644 --- a/legendary/cli.py +++ b/legendary/cli.py @@ -1029,7 +1029,7 @@ class LegendaryCLI: postinstall = self.core.install_game(igame) if postinstall: - self._handle_postinstall(postinstall, igame, yes=args.yes) + self._handle_postinstall(postinstall, igame, skip_prereqs=args.yes) dlcs = self.core.get_dlc_for_game(game.app_name) if dlcs and not args.skip_dlcs: @@ -1083,13 +1083,13 @@ class LegendaryCLI: logger.info(f'Finished installation process in {end_t - start_t:.02f} seconds.') - def _handle_postinstall(self, postinstall, igame, yes=False): + def _handle_postinstall(self, postinstall, igame, skip_prereqs=False): print('\nThis game lists the following prequisites to be installed:') print(f'- {postinstall["name"]}: {" ".join((postinstall["path"], postinstall["args"]))}') print('') if os.name == 'nt': - if yes: + if skip_prereqs: c = 'n' # we don't want to launch anything, just silent install. else: choice = input('Do you wish to install the prerequisites? ([y]es, [n]o, [i]gnore): ')