Convert kconfig_xml_fixup to python 3

No changes were necessary, but since I noticed "die"
wasn't defined I removed usages of it.

Test: python3 ./tools/kconfig_xml_fixup.py --input android-5.15/android-base-conditional.xml --output-matrix ./outputmatrix --output-version ./outputversion
Bug: 203436762
Change-Id: I7a94c57c88b96aba393655151cc5e872953dee6e
This commit is contained in:
Cole Faust 2022-04-12 14:07:14 -07:00
parent a69f1632c4
commit 9cb0c67c72
2 changed files with 32 additions and 48 deletions

View file

@ -10,13 +10,4 @@ python_binary_host {
srcs: [
"kconfig_xml_fixup.py",
],
version: {
py2: {
enabled: true,
},
py3: {
enabled: false,
},
},
}

View file

@ -1,4 +1,4 @@
#!/usr/bin/env python
#!/usr/bin/env python3
# The format of the kernel configs in the framework compatibility matrix
# has a couple properties that would make it confusing or cumbersome to
@ -24,47 +24,40 @@ import re
import sys
def fixup(args):
source_f = open(args.input) or die ("Could not open %s" % args.input)
# The first line of the conditional xml has the tag containing
# the kernel min LTS version.
line = source_f.readline()
exp_re = re.compile(r"^<kernel minlts=\"(\d+).(\d+).(\d+)\"\s+/>")
exp_match = re.match(exp_re, line)
assert exp_match, "Malformatted kernel conditional config file.\n"
major = exp_match.group(1)
minor = exp_match.group(2)
tiny = exp_match.group(3)
if args.output_version:
version_f = (open(args.output_version, "w+") or
die("Could not open version file"))
version_f.write("{}.{}.{}".format(major, minor, tiny))
version_f.close()
if args.output_matrix:
dest_f = (open(args.output_matrix, "w+") or
die("Could not open destination file"))
dest_f.write("<compatibility-matrix version=\"1.0\" type=\"framework\">\n")
# First <kernel> must not have <condition> for libvintf backwards compatibility.
dest_f.write("<kernel version=\"{}.{}.{}\" />".format(major, minor, tiny))
with open(args.input) as source_f:
# The first line of the conditional xml has the tag containing
# the kernel min LTS version.
line = source_f.readline()
while line:
line = line.replace("<value type=\"bool\">",
"<value type=\"tristate\">")
line = line.replace("<group>",
"<kernel version=\"{}.{}.{}\">".format(major, minor, tiny))
line = line.replace("</group>", "</kernel>")
dest_f.write(line)
line = source_f.readline()
exp_re = re.compile(r"^<kernel minlts=\"(\d+).(\d+).(\d+)\"\s+/>")
exp_match = re.match(exp_re, line)
assert exp_match, "Malformatted kernel conditional config file.\n"
dest_f.write("</compatibility-matrix>")
dest_f.close()
major = exp_match.group(1)
minor = exp_match.group(2)
tiny = exp_match.group(3)
source_f.close()
if args.output_version:
with open(args.output_version, "w+") as version_f:
version_f.write("{}.{}.{}".format(major, minor, tiny))
if args.output_matrix:
with open(args.output_matrix, "w+") as dest_f:
dest_f.write("<compatibility-matrix version=\"1.0\" type=\"framework\">\n")
# First <kernel> must not have <condition> for libvintf backwards compatibility.
dest_f.write("<kernel version=\"{}.{}.{}\" />".format(major, minor, tiny))
line = source_f.readline()
while line:
line = line.replace("<value type=\"bool\">",
"<value type=\"tristate\">")
line = line.replace("<group>",
"<kernel version=\"{}.{}.{}\">".format(major, minor, tiny))
line = line.replace("</group>", "</kernel>")
dest_f.write(line)
line = source_f.readline()
dest_f.write("</compatibility-matrix>")
if __name__ == '__main__':
parser = argparse.ArgumentParser(description=__doc__)