diff options
author | Damien Lespiau <damien.lespiau@intel.com> | 2015-05-19 15:04:03 +0100 |
---|---|---|
committer | Damien Lespiau <damien.lespiau@intel.com> | 2015-05-19 15:15:05 +0100 |
commit | c0a0725fad11bb7911420cf9b49f8a2ccdf35071 (patch) | |
tree | d1758f2319248498ba42a0b384c28754f86e831d /scripts/list-workarounds | |
parent | 554aac5f3e349cb5e6cd99f42826893abf564d85 (diff) |
list-workarounds: Print the line where the parsing error occured
Useful to understand the warnings the scripts prints.
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Diffstat (limited to 'scripts/list-workarounds')
-rwxr-xr-x | scripts/list-workarounds | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/scripts/list-workarounds b/scripts/list-workarounds index 42af6a34..d11b6a9d 100755 --- a/scripts/list-workarounds +++ b/scripts/list-workarounds @@ -19,10 +19,11 @@ def find_nth(haystack, needle, n): valid_platforms = ('ctg', 'elk', 'ilk', 'snb', 'ivb', 'vlv', 'hsw', 'bdw', 'chv', 'skl', 'bxt') -def parse_platforms(p): +def parse_platforms(line, p): l = p.split(',') for p in l: if p not in valid_platforms: + sys.stdout.write("warning: %s\n" % line) sys.stdout.write("unknown platform %s\n" % p) return l @@ -40,6 +41,7 @@ def parse(me): # no platform has been specified name = waname_re.search(line).group('name') path = line[:find_nth(line, ':', 2)] + sys.stdout.write("warning: %s\n" % line) sys.stdout.write("%s: no platform for %s\n" % (path, name)) continue @@ -48,12 +50,12 @@ def parse(me): platforms = match.group('platforms') if wa_name in workarounds: - platforms = parse_platforms(platforms) + platforms = parse_platforms(line, platforms) for p in platforms: if not p in workarounds[wa_name]: workarounds[wa_name].append(p) else: - workarounds[wa_name] = parse_platforms(platforms) + workarounds[wa_name] = parse_platforms(line, platforms) def execute(cmd): |