From e17c37d71167166d4e32a4f5204d032455b9a0e8 Mon Sep 17 00:00:00 2001 From: Paul Van Eck Date: Wed, 2 Aug 2023 15:06:52 -0700 Subject: [PATCH] [Perf][Python] Don't install package in editable mode The package doesn't need to be in editable mode, and we found that this can cause issues when dev dependencies that share the same namespace are installed alongside the main package. Signed-off-by: Paul Van Eck --- tools/perf-automation/Azure.Sdk.Tools.PerfAutomation/Python.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf-automation/Azure.Sdk.Tools.PerfAutomation/Python.cs b/tools/perf-automation/Azure.Sdk.Tools.PerfAutomation/Python.cs index 5dd754a89c3..06b05610e28 100644 --- a/tools/perf-automation/Azure.Sdk.Tools.PerfAutomation/Python.cs +++ b/tools/perf-automation/Azure.Sdk.Tools.PerfAutomation/Python.cs @@ -57,7 +57,7 @@ public class Python : LanguageBase { if (packageName == primaryPackage) { - await Util.RunAsync(pip, "install -e .", projectDirectory, outputBuilder: outputBuilder, errorBuilder: errorBuilder); + await Util.RunAsync(pip, "install .", projectDirectory, outputBuilder: outputBuilder, errorBuilder: errorBuilder); } // TODO: Consider installing source versions of non-primary packages. Would require finding package in source tree. // So far, this seems unnecessary, since dev-requirements.txt usually includes core.