[FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

James Darnley james.darnley at gmail.com
Tue Sep 29 22:49:22 CEST 2015


On 2015-09-29 21:56, Clément Bœsch wrote:
> On Tue, Sep 29, 2015 at 09:21:53PM +0200, Hendrik Leppkes wrote:
>> I agree, we have patchcheck for typo checking.
> 
> A lot of people do not run patchcheck (I personally never do, and given
> that we fix typo on a regular basis I'm probably not the only one in this
> situation); I think having it in FATE makes sense from a usage widespread
> PoV. But it might be relevant to drop the typo code from patchcheck then
> to avoid the duplication.

People could have it run as a post-commit git hook, then people can't
forget it (but it can be quite noisy).


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 603 bytes
Desc: OpenPGP digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150929/5a578cff/attachment.sig>


More information about the ffmpeg-devel mailing list