SL-18837: Force llprocess_test and llleap_test to use just 'python'.

On GitHub Windows runners, trying to make build.yaml set PYTHON=python in the
environment doesn't work: integration tests still fail with "Access is denied"
because they're still trying to execute the interpreter's full pathname.

Instead, make llprocess_test and llleap_test detect the case of GitHub Windows
and override the environment variable PYTHON with a baked-in string constant
"python".

1 files changed, 10 insertions, 1 deletions

diff --git a/indra/llcommon/tests/llleap_test.cpp b/indra/llcommon/tests/llleap_test.cpp
index e9edd165df..01515ecebf 100644
--- a/indra/llcommon/tests/llleap_test.cpp
+++ b/indra/llcommon/tests/llleap_test.cpp
@@ -193,11 +193,20 @@ namespace tut
                               reader.getName().substr(0, reader.getName().length()-3))),
             PYTHON(LLStringUtil::getenv("PYTHON"))
         {
+#if LL_WINDOWS
+            // Weirdly, on GitHub Windows runners, plain 'python' works much
+            // better than a full pathname.
+            const char* RUNNER_TEMP = getenv("RUNNER_TEMP");
+            if (RUNNER_TEMP && *RUNNER_TEMP)
+            {
+                PYTHON = "python";
+            }
+#endif
             ensure("Set PYTHON to interpreter pathname", !PYTHON.empty());
         }
         NamedExtTempFile reader;
         const std::string reader_module;
-        const std::string PYTHON;
+        std::string PYTHON;
     };
     typedef test_group<llleap_data> llleap_group;
     typedef llleap_group::object object;