intel/genxml: Ignore tail leading/trailing whitespace in node_validator()
authorJordan Justen <jordan.l.justen@intel.com>
Tue, 15 Aug 2023 22:26:34 +0000 (15:26 -0700)
committerMarge Bot <emma+marge@anholt.net>
Wed, 6 Sep 2023 06:51:48 +0000 (06:51 +0000)
When importing or flattening genxml with the genxml_import.py script
in MR !20593, it can lead to the tail portion of xml items differing
in whitespace.

If we strip the trailing and leading whitespace from the tail string,
and the strings are equivalent, then we can consider the xml items to
be equivalent.

Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24903>

src/intel/genxml/intel_genxml.py

index 64a7a79..21fe159 100755 (executable)
@@ -105,7 +105,7 @@ def node_validator(old: et.Element, new: et.Element) -> bool:
         # Check that the attributes are the same
         old.tag == new.tag and
         old.text == new.text and
-        old.tail == new.tail and
+        (old.tail or "").strip() == (new.tail or "").strip() and
         list(old.attrib.items()) == list(new.attrib.items()) and
         len(old) == len(new) and