On 3/18/20 6:02 PM, Vladimir Sementsov-Ogievskiy wrote:
- simplebench.py is not for executing by itself, so drop the header
- in bench_block_job.py fix python to python3
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>
---
scripts/simplebench/bench_block_job.py | 2 +-
scripts/simplebench/simplebench.py | 2 --
2 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/scripts/simplebench/bench_block_job.py
b/scripts/simplebench/bench_block_job.py
index 9808d696cf..a0dda1dc4e 100755
--- a/scripts/simplebench/bench_block_job.py
+++ b/scripts/simplebench/bench_block_job.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
#
# Benchmark block jobs
#
diff --git a/scripts/simplebench/simplebench.py
b/scripts/simplebench/simplebench.py
index 59e7314ff6..7e25f3590b 100644
--- a/scripts/simplebench/simplebench.py
+++ b/scripts/simplebench/simplebench.py
@@ -1,5 +1,3 @@
-#!/usr/bin/env python
-#
# Simple benchmarking framework
#
# Copyright (c) 2019 Virtuozzo International GmbH.
I'd rather see this squashed in patches 1 and 2, if not:
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
