-#!/usr/bin/env python
+#!/usr/bin/env python2
#
# Copyright (c) 2016 ARM Limited
# All rights reserved
#
# Authors: Andreas Sandberg
+from tempfile import TemporaryFile
import os
+import subprocess
import sys
from style.repo import GitRepo
os.chdir(repo_base)
failing_files = set()
+staged_mismatch = set()
for status, fname in git.status(filter="MA", cached=True):
if args.verbose:
else:
regions = all_regions
+ # Show they appropriate object and dump it to a file
+ status = git.file_from_index(fname)
+ f = TemporaryFile()
+ f.write(status)
+
verifiers = [ v(ui, opts, base=repo_base) for v in all_verifiers ]
for v in verifiers:
- if not v.skip(fname) and v.check(fname, regions):
+ f.seek(0)
+ # It is prefered that the first check is silent as it is in the
+ # staged file. If the check fails, then we will do it non-silently
+ # on the current file, reporting meaningful shortcomings
+ if not v.skip(fname) and v.check(fname, regions, fobj=f, silent=True):
failing_files.add(fname)
+ if not v.check(fname, regions):
+ staged_mismatch.add(fname)
+ f.close()
if failing_files:
- print >> sys.stderr
- print >> sys.stderr, "Style checker failed for the following files:"
- for f in failing_files:
- print >> sys.stderr, "\t%s" % f
- print >> sys.stderr
- print >> sys.stderr, \
+ if len(failing_files) > len(staged_mismatch):
+ print >> sys.stderr
+ print >> sys.stderr, "Style checker failed for the following files:"
+ for f in failing_files:
+ if f not in staged_mismatch:
+ print >> sys.stderr, "\t%s" % f
+ print >> sys.stderr
+ print >> sys.stderr, \
"Please run the style checker manually to fix the offending files.\n" \
"To check your modifications, run: util/style.py -m"
+
+ print >> sys.stderr
+ if staged_mismatch:
+ print >> sys.stderr, \
+ "It looks like you have forgotten to stage your fixes for commit in\n"\
+ "the following files: "
+ for f in staged_mismatch:
+ print >> sys.stderr, "\t%s" % f
+ print >> sys.stderr, "Please `git --add' them"
sys.exit(1)