[coreboot] How to best integrate misspelling script?

[Paul Menzel]

Dear coreboot folks,


searching for a way to automatically find misspellings I found a post by
Jim Meyering on Fedora’s devel list [1], where he suggests to use the
project misspellings [2].

Testing this with a simple

    $ find src/coreboot/ -name '*.c' | ./misspellings -f -

it indeed finds a lot of misspelling in our code, but as expected there
are also false positives. I am going to send in a patch with the fixes
after reviewing the list to omit the false positives.

The question is, how to hook this up into our infrastructure. Jim
suggests a `make check` rule. The downside is that this would need to be
run manually by the user. Running it automatically on every commit by
placing a script in `util/lint` to only check changed files would be
feasible, but possible false positives are a problem though.

Do you have an idea, how to work around this problem? There is the
`--no-verify` switch for `git commit` to not run the hooks, but this
also ignores the commit-msg hooks, which then is not able to add a
Change-Id for Gerrit.


Thanks,

Paul


[1] https://lists.fedoraproject.org/pipermail/devel/2012-June/168159.html
[2] http://github.com/lyda/misspell-check
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20130629/50a94c4f/attachment.sig>