From c287649e07f8c5f791c48d448facc33b01c66201 Mon Sep 17 00:00:00 2001 From: derrod Date: Thu, 30 Apr 2020 12:31:23 +0200 Subject: [PATCH] [cli] Make app name optional for "list-files" If reading from a file/url the app name is not necessary. --- legendary/cli.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/legendary/cli.py b/legendary/cli.py index 6405ec5..c4d8185 100644 --- a/legendary/cli.py +++ b/legendary/cli.py @@ -150,6 +150,10 @@ class LegendaryCLI: if args.platform_override: args.force_download = True + if not args.override_manifest and not args.app_name: + print('You must provide either a manifest url/path or app name!') + return + # check if we even need to log in if args.override_manifest: logger.info(f'Loading manifest from "{args.override_manifest}"') @@ -412,7 +416,7 @@ def main(): install_parser.add_argument('app_name', help='Name of the app', metavar='') uninstall_parser.add_argument('app_name', help='Name of the app', metavar='') launch_parser.add_argument('app_name', help='Name of the app', metavar='') - list_files_parser.add_argument('app_name', help='Name of the app', metavar='') + list_files_parser.add_argument('app_name', nargs='?', help='Name of the app', metavar='') # importing only works on Windows right now if os.name == 'nt':