From f5fb293295241e64d745ed63da5c603eed0ebc22 Mon Sep 17 00:00:00 2001
From: Joshua Nelson <jnelson@cloudflare.com>
Date: Tue, 29 Mar 2022 13:50:17 -0500
Subject: [PATCH] Clarify how `src/tools/x` searches for python

---
 src/tools/x/src/main.rs | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/tools/x/src/main.rs b/src/tools/x/src/main.rs
index 8c47559b36939..57d548f313dae 100644
--- a/src/tools/x/src/main.rs
+++ b/src/tools/x/src/main.rs
@@ -26,6 +26,7 @@ fn python() -> &'static str {
     let mut python3 = false;
 
     for dir in env::split_paths(&val) {
+        // `python` should always take precedence over python2 / python3 if it exists
         if dir.join(PYTHON).exists() {
             return PYTHON;
         }
@@ -34,11 +35,14 @@ fn python() -> &'static str {
         python3 |= dir.join(PYTHON3).exists();
     }
 
+    // try 3 before 2
     if python3 {
         PYTHON3
     } 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
     }
 }