about summary refs log tree commit diff
diff options
context:
space:
mode:
authorMatthias Krüger <matthias.krueger@famsik.de>2022-08-01 16:49:35 +0200
committerGitHub <noreply@github.com>2022-08-01 16:49:35 +0200
commit058ea7a955982a7bcb3ba72c1b644558241c96e4 (patch)
tree6206e4725b190c9ed4bc9cedb3923703fbccf4d5
parent4606830f83ea8db9bc8f7861e8884c90b806beac (diff)
parentfcf1f958aca7c46939911f52a0093fca7e5c2a80 (diff)
Rollup merge of #100020 - CallumIO:issue-99648, r=jyn514
better error when python is not found in x - issue #99648

`x` now shows an appropriate error message and exits, when a version of `python` is not found on the users `PATH`.

Resolves #99648
-rw-r--r--src/tools/x/src/main.rs6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/tools/x/src/main.rs b/src/tools/x/src/main.rs
index 57d548f313d..9187c3551d7 100644
--- a/src/tools/x/src/main.rs
+++ b/src/tools/x/src/main.rs
@@ -41,9 +41,9 @@ fn python() -> &'static str {
     } else if python2 {
         PYTHON2
     } else {
-        // We would have returned early if we found that python is installed ...
-        // maybe this should panic with an error instead?
-        PYTHON
+        // Python was not found on path, so exit
+        eprintln!("Unable to find python in your PATH. Please check it is installed.");
+        process::exit(1);
     }
 }