[Lustre-devel] [wc-discuss] Lustre code style Git commit hooks and

Andreas Dilger adilger at whamcloud.com
Wed Nov 9 09:04:58 PST 2011

On 2011-11-09, at 9:27 AM, James Simmons <jsimmons at infradead.org> wrote:
> Speaking of I just attempted to push a patch and noticed that no Change-Id 
> is being generated when I commit a new patch. This is with the default 
> hooks that I copied form build directory.

Is this from the tip of the master branch?

An earlier version of the commit-msg hook used "git hash-object -t commit" and it caused an empty Change-Id string for older versions of Git. The "-t commit" option was removed from the latest version and solved the problem for the people that used it.

If updating to the latest version (or removing "-t commit") my fix this for you, please file a bug assigned to me and we can work it out. 

The contents of the .git directory are not under revision control, so it isn't possible to automatically update the commit hooks. At some point I'd like to update the hooks via "make", but was holding off until I have more confidence that issues like this related to different development environments are fixed. 

Thanks for the feedback. 

Cheers, Andreas

More information about the lustre-devel mailing list