[PATCH v1] perf test python: Put perf python at start of sys.path

Ian Rogers posted 1 patch 2 years, 8 months ago
tools/perf/tests/python-use.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH v1] perf test python: Put perf python at start of sys.path
Posted by Ian Rogers 2 years, 8 months ago
This avoids picking up a system installed version of the perf python module.

Signed-off-by: Ian Rogers <irogers@google.com>
---
 tools/perf/tests/python-use.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/tests/python-use.c b/tools/perf/tests/python-use.c
index 6b990ee38575..0ebc22ac8d5b 100644
--- a/tools/perf/tests/python-use.c
+++ b/tools/perf/tests/python-use.c
@@ -14,7 +14,7 @@ static int test__python_use(struct test_suite *test __maybe_unused, int subtest
 	char *cmd;
 	int ret;
 
-	if (asprintf(&cmd, "echo \"import sys ; sys.path.append('%s'); import perf\" | %s %s",
+	if (asprintf(&cmd, "echo \"import sys ; sys.path.insert(0, '%s'); import perf\" | %s %s",
 		     PYTHONPATH, PYTHON, verbose > 0 ? "" : "2> /dev/null") < 0)
 		return -1;
 
-- 
2.41.0.rc0.172.g3f132b7071-goog
Re: [PATCH v1] perf test python: Put perf python at start of sys.path
Posted by Arnaldo Carvalho de Melo 2 years, 8 months ago
Em Fri, May 26, 2023 at 10:55:17PM -0700, Ian Rogers escreveu:
> This avoids picking up a system installed version of the perf python module.

Thanks, applied.

- Arnaldo

 
> Signed-off-by: Ian Rogers <irogers@google.com>
> ---
>  tools/perf/tests/python-use.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/perf/tests/python-use.c b/tools/perf/tests/python-use.c
> index 6b990ee38575..0ebc22ac8d5b 100644
> --- a/tools/perf/tests/python-use.c
> +++ b/tools/perf/tests/python-use.c
> @@ -14,7 +14,7 @@ static int test__python_use(struct test_suite *test __maybe_unused, int subtest
>  	char *cmd;
>  	int ret;
>  
> -	if (asprintf(&cmd, "echo \"import sys ; sys.path.append('%s'); import perf\" | %s %s",
> +	if (asprintf(&cmd, "echo \"import sys ; sys.path.insert(0, '%s'); import perf\" | %s %s",
>  		     PYTHONPATH, PYTHON, verbose > 0 ? "" : "2> /dev/null") < 0)
>  		return -1;
>  
> -- 
> 2.41.0.rc0.172.g3f132b7071-goog
> 

-- 

- Arnaldo