[PATCH i-g-t v2 6/6] tools/intel-gfx-fw-info: do not use bare 'except'

Lucas De Marchi lucas.demarchi at intel.com
Tue Aug 20 23:29:28 UTC 2024


>From flake8:

	tools/intel-gfx-fw-info:137:1: E722 do not use bare 'except'

Now it passes clean with --ignore=E101,E501,W191 - those ignores are
needed because they are triggering in the C string we use, CDEF, and
flake8 can't bother enabling/disabling warnings per block.

Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
---
 tools/intel-gfx-fw-info | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/intel-gfx-fw-info b/tools/intel-gfx-fw-info
index cbabda913..3c9b47809 100755
--- a/tools/intel-gfx-fw-info
+++ b/tools/intel-gfx-fw-info
@@ -133,11 +133,11 @@ logging.basicConfig(format="%(levelname)s: %(message)s")
 
 try:
     from dissect import cstruct
-except:
+except ImportError as e:
     logging.critical(
         "Could not import dissect.cstruct module. See https://github.com/fox-it/dissect.cstruct for installation options"
     )
-    raise SystemExit(1)
+    raise SystemExit(1) from e
 
 
 GSC_CPD_ENTRY_OFFSET_MASK = 0x1FFFFFF
-- 
2.43.0



More information about the igt-dev mailing list