From e2fa3aeae40f2ba33a6689a4933cef8480f55dbb Mon Sep 17 00:00:00 2001 From: Paul Van Eck Date: Thu, 3 Aug 2023 12:03:05 -0700 Subject: [PATCH] [Perf][Python] Don't install package in editable mode (#6652) 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.