buildtools: fix AVX512 check for Python 3.5
authorLance Richardson <lance.richardson@broadcom.com>
Mon, 3 Jan 2022 17:09:40 +0000 (12:09 -0500)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 12 Jan 2022 17:06:52 +0000 (18:06 +0100)
Python 3.5 subprocess.run() does not have a capture_output
parameter (it is present only in 3.7 and up).
Capture output by using subprocess.PIPE for stdout instead.

Fixes: bb9cd91095b3 ("buildtools: make AVX512 check portable")
Cc: stable@dpdk.org
Signed-off-by: Lance Richardson <lance.richardson@broadcom.com>
Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
buildtools/binutils-avx512-check.py

index 57392ec..a0847a2 100644 (file)
@@ -15,7 +15,7 @@ with tempfile.NamedTemporaryFile() as obj:
     src = '__asm__("vpgatherqq {}");'.format(gather_params).encode('utf-8')
     subprocess.run(cc + ['-c', '-xc', '-o', obj.name, '-'], input=src, check=True)
     asm = subprocess.run([objdump, '-d', '--no-show-raw-insn', obj.name],
-                         capture_output=True, check=True).stdout.decode('utf-8')
+                         stdout=subprocess.PIPE, check=True).stdout.decode('utf-8')
     if gather_params not in asm:
            print('vpgatherqq displacement error with as')
            sys.exit(1)