From 5e4e86daa26927d1dddc5a2bb5031891d664c13e Mon Sep 17 00:00:00 2001 From: Eirini Koutsaniti Date: Thu, 28 Nov 2019 08:48:22 +0100 Subject: [PATCH 1/3] Improve error message when no action is specified --- reframe/frontend/cli.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/reframe/frontend/cli.py b/reframe/frontend/cli.py index 690c27e856..e4b7ea87b8 100644 --- a/reframe/frontend/cli.py +++ b/reframe/frontend/cli.py @@ -617,7 +617,8 @@ def main(): printer.info(runner.stats.performance_report()) else: - printer.info('No action specified. Exiting...') + printer.info("No action specified. Please specify '-l' for " + "listing or '-r' for running.") printer.info("Try `%s -h' for a list of available actions." % argparser.prog) sys.exit(1) From 7124185d40dcb00e6d199e58362a01fae2032b17 Mon Sep 17 00:00:00 2001 From: Eirini Koutsaniti Date: Mon, 2 Dec 2019 16:32:27 +0100 Subject: [PATCH 2/3] Fix quotes --- reframe/frontend/cli.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/reframe/frontend/cli.py b/reframe/frontend/cli.py index e4b7ea87b8..2ddde91159 100644 --- a/reframe/frontend/cli.py +++ b/reframe/frontend/cli.py @@ -617,8 +617,8 @@ def main(): printer.info(runner.stats.performance_report()) else: - printer.info("No action specified. Please specify '-l' for " - "listing or '-r' for running.") + printer.info("No action specified. Please specify `-l'/`-L' for " + "listing or `-r' for running.") printer.info("Try `%s -h' for a list of available actions." % argparser.prog) sys.exit(1) From f9f5ae3a89ae59a5df804d3aa3e43f6e05f988a8 Mon Sep 17 00:00:00 2001 From: Vasileios Karakasis Date: Mon, 2 Dec 2019 21:11:55 +0100 Subject: [PATCH 3/3] Fine tune error message --- reframe/frontend/cli.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/reframe/frontend/cli.py b/reframe/frontend/cli.py index 2ddde91159..c3a1a0815d 100644 --- a/reframe/frontend/cli.py +++ b/reframe/frontend/cli.py @@ -617,10 +617,10 @@ def main(): printer.info(runner.stats.performance_report()) else: - printer.info("No action specified. Please specify `-l'/`-L' for " - "listing or `-r' for running.") - printer.info("Try `%s -h' for a list of available actions." % - argparser.prog) + printer.error("No action specified. Please specify `-l'/`-L' for " + "listing or `-r' for running. " + "Try `%s -h' for more options." % + argparser.prog) sys.exit(1) if not success: