Mesa (staging/20.0): intel: fix gen_sort_tags.py
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Tue Jun 9 18:01:49 UTC 2020
Module: Mesa
Branch: staging/20.0
Commit: 3f0a6cad4d768a132573bc2107b861b773810018
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3f0a6cad4d768a132573bc2107b861b773810018
Author: Eric Engestrom <eric at engestrom.ch>
Date: Fri Jun 5 11:26:56 2020 +0200
intel: fix gen_sort_tags.py
The script was failing for me (python 3.8), not sure if this is a recent
python version break or not as I don't know how often people have been
running this script:
Processing ./gen9.xml... Traceback (most recent call last):
File "./gen_sort_tags.py", line 177, in <module>
main()
File "./gen_sort_tags.py", line 170, in main
genxml[:] = enums + sorted_structs.values() + instructions + registers
TypeError: can only concatenate list (not "odict_values") to list
Turning the odict into a list fixes it for me, and the resulting xml
file are identical to before :)
Fixes: 903e142f0d35bc550ffd ("genxml: add a sorting script")
Signed-off-by: Eric Engestrom <eric at engestrom.ch>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5352>
(cherry picked from commit 981d07c74a1611d8c308a96f59899fff66674c1a)
---
.pick_status.json | 2 +-
src/intel/genxml/gen_sort_tags.py | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/.pick_status.json b/.pick_status.json
index 395a7615e33..3f93e35be1a 100644
--- a/.pick_status.json
+++ b/.pick_status.json
@@ -643,7 +643,7 @@
"description": "intel: fix gen_sort_tags.py",
"nominated": true,
"nomination_type": 1,
- "resolution": 0,
+ "resolution": 1,
"master_sha": null,
"because_sha": "903e142f0d35bc550ffde321987a5b6fca1095eb"
},
diff --git a/src/intel/genxml/gen_sort_tags.py b/src/intel/genxml/gen_sort_tags.py
index 66b1011cef0..ea6b6d4158b 100644
--- a/src/intel/genxml/gen_sort_tags.py
+++ b/src/intel/genxml/gen_sort_tags.py
@@ -167,7 +167,7 @@ def main():
for r in registers:
r[:] = sorted(r.getchildren(), key=get_start)
- genxml[:] = enums + sorted_structs.values() + instructions + registers
+ genxml[:] = enums + list(sorted_structs.values()) + instructions + registers
print('<?xml version="1.0" ?>')
print_node(sys.stdout, 0, genxml)
More information about the mesa-commit
mailing list