[20.08,1/9] app/test-bbdev: support python3 only
diff mbox series

Message ID 20200522132320.26373-2-louise.kilheeney@intel.com
State Superseded
Delegated to: Thomas Monjalon
Headers show
Series
  • adding support for python 3 only.
Related show

Checks

Context Check Description
ci/Intel-compilation success Compilation OK
ci/iol-testing fail Testing issues
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-nxp-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/checkpatch success coding style OK

Commit Message

Kilheeney, Louise May 22, 2020, 1:23 p.m. UTC
Changed script to explicitly use python3 only.

Cc: Nicolas Chautru <nicolas.chautru@intel.com>

Signed-off-by: Louise Kilheeney <louise.kilheeney@intel.com>
---
 app/test-bbdev/test-bbdev.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

Comments

Nicolas Chautru May 26, 2020, 6:58 p.m. UTC | #1
Kilheeney, Louise <louise.kilheeney@intel.com> :
> Sent: Friday, May 22, 2020 6:23 AM
> To: dev@dpdk.org
> Cc: Kilheeney, Louise <louise.kilheeney@intel.com>; Chautru, Nicolas
> <nicolas.chautru@intel.com>
> Subject: [PATCH 20.08 1/9] app/test-bbdev: support python3 only
> 
> Changed script to explicitly use python3 only.
> 
> Cc: Nicolas Chautru <nicolas.chautru@intel.com>
> 
> Signed-off-by: Louise Kilheeney <louise.kilheeney@intel.com>
> ---
>  app/test-bbdev/test-bbdev.py | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/app/test-bbdev/test-bbdev.py b/app/test-bbdev/test-bbdev.py
> index 0194be046..50add4580 100755
> --- a/app/test-bbdev/test-bbdev.py
> +++ b/app/test-bbdev/test-bbdev.py
> @@ -1,5 +1,4 @@
> -#!/usr/bin/env python
> -
> +#!/usr/bin/env python3
>  # SPDX-License-Identifier: BSD-3-Clause  # Copyright(c) 2017 Intel
> Corporation
> 
> @@ -12,7 +11,7 @@
>  from threading import Timer
> 
>  def kill(process):
> -    print "ERROR: Test app timed out"
> +    print ("ERROR: Test app timed out")
>      process.kill()
> 
>  if "RTE_SDK" in os.environ:
> @@ -66,7 +65,7 @@ def kill(process):
>  args = parser.parse_args()
> 
>  if not os.path.exists(args.testapp_path):
> -    print "No such file: " + args.testapp_path
> +    print ("No such file: " + args.testapp_path)
>      sys.exit(1)
> 
>  params = [args.testapp_path]
> --
> 2.17.1

I double checked locally, fine with me. Thanks. 

Acked-by: Nic Chautru

Patch
diff mbox series

diff --git a/app/test-bbdev/test-bbdev.py b/app/test-bbdev/test-bbdev.py
index 0194be046..50add4580 100755
--- a/app/test-bbdev/test-bbdev.py
+++ b/app/test-bbdev/test-bbdev.py
@@ -1,5 +1,4 @@ 
-#!/usr/bin/env python
-
+#!/usr/bin/env python3
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2017 Intel Corporation
 
@@ -12,7 +11,7 @@ 
 from threading import Timer
 
 def kill(process):
-    print "ERROR: Test app timed out"
+    print ("ERROR: Test app timed out")
     process.kill()
 
 if "RTE_SDK" in os.environ:
@@ -66,7 +65,7 @@  def kill(process):
 args = parser.parse_args()
 
 if not os.path.exists(args.testapp_path):
-    print "No such file: " + args.testapp_path
+    print ("No such file: " + args.testapp_path)
     sys.exit(1)
 
 params = [args.testapp_path]