diff options
| author | Markus Armbruster | 2015-06-09 18:32:29 +0200 |
|---|---|---|
| committer | Markus Armbruster | 2015-06-18 14:12:34 +0200 |
| commit | 8608d2525186062099a38971c276752e7a38903a (patch) | |
| tree | 504f996ece17d982bd4c8e72b6beae284be81ef0 /scripts | |
| parent | qapi: Improve a couple of confusing variable names (diff) | |
| download | qemu-8608d2525186062099a38971c276752e7a38903a.tar.gz qemu-8608d2525186062099a38971c276752e7a38903a.tar.xz qemu-8608d2525186062099a38971c276752e7a38903a.zip | |
qapi: Fix file name in error messages for included files
We print the name as it appears in the include expression. Tools
processing error messages want it relative to the working directory.
Make it so.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/qapi.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/scripts/qapi.py b/scripts/qapi.py index c2eb12ba3a..716e348a3c 100644 --- a/scripts/qapi.py +++ b/scripts/qapi.py @@ -101,14 +101,13 @@ class QAPIExprError(Exception): class QAPISchema: - def __init__(self, fp, fname = None, include_hist = [], + def __init__(self, fp, include_hist = [], previously_included = [], incl_info = None): """ include_hist is a stack used to detect inclusion cycles previously_included is a global state used to avoid multiple inclusions of the same file""" abs_fname = os.path.abspath(fp.name) - if fname is None: - fname = fp.name + fname = fp.name self.fname = fname self.include_hist = include_hist + [(fname, abs_fname)] previously_included.append(abs_fname) @@ -148,7 +147,7 @@ class QAPISchema: except IOError, e: raise QAPIExprError(expr_info, '%s: %s' % (e.strerror, include)) - exprs_include = QAPISchema(fobj, include, self.include_hist, + exprs_include = QAPISchema(fobj, self.include_hist, previously_included, expr_info) self.exprs.extend(exprs_include.exprs) else: |