[Python-Dev] whitespace normalization pre-commit hook is giving me grief

[Scott Dial]

Martin v. Löwis wrote:
> Brett Cannon schrieb:
>> Unfortunately the pre-commit hook
>> does not specify what line a change was made on so I have no clue where
>> it is failing (maybe this should be added?).
> 
> It creates a reindent.Reindenter on the new contents, then invokes
> .run() on it, and complains if that returns true. If you can come
> up with a patch to reindent that makes it report more detailed
> errors, please post it, and I'll try to merge it into the hook
> script.
> 

How about you do:

     if reindenter.run():
         print >>sys.stderr, "file %s is not whitespace-normalized" %path
	print >>sys.stderr, difflib.unified_diff(reindenter.raw,
                                                  reindenter.after)
         bad += 1

Which would provide you a unified diff to give you a clue.

-- 
Scott Dial
scott at scottdial.com
scodial at cs.indiana.edu