On Wed, 18 Jan 2012 20:42:43 +0100, Chase Douglas <[email protected]> wrote:
> 1. Should we just go with linux style, so we can use check_patch without > any modifications? > > 2. If not, will we fork check_patch or make our own? I'd be good with either plan. I can also run check_patch on anything getting added to the server, with suitable nag mails for patches which don't pass. > 3. Who will enforce style? I believe if we have a script we can add a > commit hook to the git server, but that may be too heavy-handed. It wouldn't be hard for me to make that part of the regular patch merging process > 4. If no one will enforce style, do we want to periodically run indent > to fix things. Maybe once per cycle after the merge window closes (as > you suggest here, but I begged for mercy against this time :)? No, once will be painful enough. After that, all new patches would be required to follow the formatting conventions or not get accepted. Keeping the code correct at all times makes it easier to fix new code -- just re-indent all changed files. -- [email protected]
pgpcvnfg4naSW.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
