util/glsl2spirv: fix appending extra flags
authorDylan Baker <dylan.c.baker@intel.com>
Tue, 1 Nov 2022 19:49:53 +0000 (12:49 -0700)
committerEric Engestrom <eric@engestrom.ch>
Thu, 17 Nov 2022 14:05:02 +0000 (14:05 +0000)
The variable is called `extra`, but what's written is `extra - flags`,
and `flags` is undefined, so if the variable was ever passed there would
be an uncaught exception.

fixes: 9786d9ef2abb45a4e832cf1347581e3ca3aae9f0

Reviewed-by: Luis Felipe Strano Moraes <luis.strano@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/19449>
(cherry picked from commit 4ffa8a9ac010f3db79b1e872f7e664a4470cb981)

.pick_status.json
src/util/glsl2spirv.py

index 0a3a90c..9697ef8 100644 (file)
         "description": "util/glsl2spirv: fix appending extra flags",
         "nominated": true,
         "nomination_type": 1,
-        "resolution": 0,
+        "resolution": 1,
         "main_sha": null,
         "because_sha": "9786d9ef2abb45a4e832cf1347581e3ca3aae9f0"
     },
index 8b765c9..a0acefd 100644 (file)
@@ -136,7 +136,7 @@ def process_file(args):
             cmd_list += ["--variable-name", args.vn]
 
         if args.extra is not None:
-            cmd_list.append(args.extra-flags)
+            cmd_list.append(args.extra)
 
         if args.create_entry is not None:
             cmd_list += ["--entry-point", args.create_entry]