@@ -68,8 +68,7 @@ compat/%.i: compat/%.c Makefile
compat/%.c: public/%.h xlat.lst Makefile $(BASEDIR)/tools/compat-build-source.py
mkdir -p $(@D)
- grep -v 'DEFINE_XEN_GUEST_HANDLE(long)' $< | \
- $(PYTHON) $(BASEDIR)/tools/compat-build-source.py >$@.new
+ $(PYTHON) $(BASEDIR)/tools/compat-build-source.py xlat.lst <$< >$@.new
mv -f $@.new $@
compat/.xlat/%.h: compat/%.h compat/.xlat/%.lst $(BASEDIR)/tools/get-fields.sh Makefile
@@ -12,7 +12,11 @@ pats = [
[ r"XEN_GUEST_HANDLE(_[0-9A-Fa-f]+)?", r"COMPAT_HANDLE" ],
];
-xlatf = open('xlat.lst', 'r')
+try:
+ xlatf = open(sys.argv[1], 'r')
+except IndexError:
+ print('missing path to xlat.lst argument')
+ sys.exit(1)
for line in xlatf.readlines():
match = re.subn(r"^\s*\?\s+(\w*)\s.*", r"\1", line.rstrip())
if match[1]:
@@ -24,6 +28,8 @@ for pat in pats:
pat[0] = re.compile(pat[0])
for line in sys.stdin.readlines():
+ if 'DEFINE_XEN_GUEST_HANDLE(long)' in line:
+ continue
for pat in pats:
line = re.sub(pat[0], pat[1], line)
print(line.rstrip())
Improvement are: - give the path to xlat.lst as argument - include `grep -v` in compat-build-source.py script, we don't need to write this in several scripted language. No changes in final compat/%.h headers. Signed-off-by: Anthony PERARD <anthony.perard@citrix.com> --- Notes: v5: - removed "have 'xlat.lst' path as a variable" from the patch. v4: - new patch xen/include/Makefile | 3 +-- xen/tools/compat-build-source.py | 8 +++++++- 2 files changed, 8 insertions(+), 3 deletions(-)