mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-07-11 12:25:52 +00:00
4a5370ba41
As part of a goal to eliminate Perl from libvirt build tools, rewrite the check-drivername.pl tool in Python. This was mostly a straight conversion, manually going line-by-line to change the syntax from Perl to Python. Thus the overall structure of the file and approach is the same. In testing though it was discovered the existing code was broken since it hadn't been updated after driver.h was split into many files. Since the old code is being thrown away, the fix was done as part of the rewrite rather than split into a separate commit. Tested-by: Cole Robinson <crobinso@redhat.com> Reviewed-by: Ján Tomko <jtomko@redhat.com> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
114 lines
3.9 KiB
Python
114 lines
3.9 KiB
Python
#!/usr/bin/env python
|
|
#
|
|
# Copyright (C) 2013-2019 Red Hat, Inc.
|
|
#
|
|
# This library is free software; you can redistribute it and/or
|
|
# modify it under the terms of the GNU Lesser General Public
|
|
# License as published by the Free Software Foundation; either
|
|
# version 2.1 of the License, or (at your option) any later version.
|
|
#
|
|
# This library is distributed in the hope that it will be useful,
|
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
# Lesser General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU Lesser General Public
|
|
# License along with this library. If not, see
|
|
# <http://www.gnu.org/licenses/>.
|
|
#
|
|
|
|
from __future__ import print_function
|
|
|
|
import re
|
|
import sys
|
|
|
|
drvfiles = []
|
|
symfiles = []
|
|
for arg in sys.argv:
|
|
if arg.endswith(".h"):
|
|
drvfiles.append(arg)
|
|
else:
|
|
symfiles.append(arg)
|
|
|
|
symbols = {}
|
|
|
|
for symfile in symfiles:
|
|
with open(symfile, "r") as fh:
|
|
for line in fh:
|
|
m = re.search(r'''^\s*(vir\w+)\s*;\s*$''', line)
|
|
if m is not None:
|
|
symbols[m.group(1)] = True
|
|
|
|
status = 0
|
|
for drvfile in drvfiles:
|
|
with open(drvfile, "r") as fh:
|
|
for line in fh:
|
|
m = re.search(r'''\*(virDrv\w+)\s*\)''', line)
|
|
if m is not None:
|
|
drv = m.group(1)
|
|
|
|
skip = [
|
|
"virDrvStateInitialize",
|
|
"virDrvStateCleanup",
|
|
"virDrvStateReload",
|
|
"virDrvStateStop",
|
|
"virDrvConnectSupportsFeature",
|
|
"virDrvConnectURIProbe",
|
|
"virDrvDomainMigratePrepare",
|
|
"virDrvDomainMigratePrepare2",
|
|
"virDrvDomainMigratePrepare3",
|
|
"virDrvDomainMigratePrepare3Params",
|
|
"virDrvDomainMigratePrepareTunnel",
|
|
"virDrvDomainMigratePrepareTunnelParams",
|
|
"virDrvDomainMigratePrepareTunnel3",
|
|
"virDrvDomainMigratePrepareTunnel3Params",
|
|
"virDrvDomainMigratePerform",
|
|
"virDrvDomainMigratePerform3",
|
|
"virDrvDomainMigratePerform3Params",
|
|
"virDrvDomainMigrateConfirm",
|
|
"virDrvDomainMigrateConfirm3",
|
|
"virDrvDomainMigrateConfirm3Params",
|
|
"virDrvDomainMigrateBegin",
|
|
"virDrvDomainMigrateBegin3",
|
|
"virDrvDomainMigrateBegin3Params",
|
|
"virDrvDomainMigrateFinish",
|
|
"virDrvDomainMigrateFinish2",
|
|
"virDrvDomainMigrateFinish3",
|
|
"virDrvDomainMigrateFinish3Params",
|
|
"virDrvStreamInData",
|
|
]
|
|
if drv in skip:
|
|
continue
|
|
|
|
sym = drv.replace("virDrv", "vir")
|
|
|
|
if sym not in symbols:
|
|
print("Driver method name %s doesn't match public API" %
|
|
drv)
|
|
status = 1
|
|
continue
|
|
|
|
m = re.search(r'''(\*vir\w+)\s*\)''', line)
|
|
if m is not None:
|
|
name = m.group(1)
|
|
print("Bogus name %s" % name)
|
|
status = 1
|
|
continue
|
|
|
|
m = re.search(r'''^\s*(virDrv\w+)\s+(\w+);\s*''', line)
|
|
if m is not None:
|
|
drv = m.group(1)
|
|
field = m.group(2)
|
|
|
|
tmp = drv.replace("virDrv", "")
|
|
if tmp.startswith("NWFilter"):
|
|
tmp = "nwfilter" + tmp[8:]
|
|
tmp = tmp[0:1].lower() + tmp[1:]
|
|
|
|
if tmp != field:
|
|
print("Driver struct field %s should be named %s" %
|
|
(field, tmp))
|
|
status = 1
|
|
|
|
sys.exit(status)
|