From 14a6c0e601b246601954ca18fb44d3cb346e331f Mon Sep 17 00:00:00 2001 From: Ole Troan Date: Wed, 13 May 2020 11:47:43 +0200 Subject: [PATCH] vppapigen: add assert for python version The recent introduction of f-strings require >=Python3.6 Fix a couple of pycodestyle warnings. Type: fix Signed-off-by: Ole Troan Change-Id: Id9e375bb8febb157d7f9d7fa21808724770df6f8 --- src/tools/vppapigen/vppapigen.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/tools/vppapigen/vppapigen.py b/src/tools/vppapigen/vppapigen.py index 28712e4d949..2d2097904c5 100755 --- a/src/tools/vppapigen/vppapigen.py +++ b/src/tools/vppapigen/vppapigen.py @@ -11,6 +11,8 @@ import os import sys from subprocess import Popen, PIPE +assert sys.version_info >= (3, 6), \ + "Not supported Python version: {}".format(sys.version) log = logging.getLogger('vppapigen') # Ensure we don't leave temporary files around @@ -285,7 +287,7 @@ class Define(): self.options[b.option] = b.value remove.append(b) - block = [x for x in block if not x in remove] + block = [x for x in block if x not in remove] self.block = block self.vla = vla_is_last_check(name, block) self.crc = str(block).encode() @@ -765,7 +767,7 @@ class VPPAPI(object): block = [Field('u32', 'context'), Field('i32', 'retval')] # inherhit the parent's options - for k,v in parent.options.items(): + for k, v in parent.options.items(): block.append(Option(k, v)) return Define(name + '_reply', [], block) -- 2.16.6