Exactly. IMHO, this is easier done committing interactively with git commit <enter> and letting your favorite editor open up.<br><br><div class="gmail_quote">On Fri, Apr 23, 2010 at 10:43 AM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I have a quick question about commit messages.  The pre-commit hook<br>
tells me that the first line of the commit message must be 78 chars or<br>
less, and the second line must be blank.<br>
<br>
So how should commit messages be structured?  A short descriptive<br>
line, followed by a blank line, followed optionally by one or two<br>
sentences giving details of the change?<br>
<font color="#888888"><br>
   David<br>
</font><div><div></div><div class="h5"><br>
<br>
On Fri, Apr 23, 2010 at 6:50 AM, Robert Maynard<br>
<<a href="mailto:robert.maynard@kitware.com">robert.maynard@kitware.com</a>> wrote:<br>
><br>
><br>
> On Fri, Apr 23, 2010 at 2:24 AM, Mark Olesen <<a href="mailto:Mark.Olesen@faurecia.com">Mark.Olesen@faurecia.com</a>><br>
> wrote:<br>
>><br>
>> On Fri, 2010-04-23 at 10:34 +1000, Andrew Maclean wrote:<br>
>> > I agree with you, I  think that there also should be a commit check<br>
>> > for tabs,  otherwise layout gets really messy.<br>
>><br>
>> What about checking/stripping trailing whitespace as well?<br>
>> This is something else that tends to look messy and result in lots of<br>
>> noisy commits.<br>
>><br>
>> /mark<br>
>><br>
>><br>
>> DISCLAIMER:<br>
>> This electronic transmission (and any attachments thereto) is intended<br>
>> solely for the use of the addressee(s). It may contain confidential or<br>
>> legally privileged information. If you are not the intended recipient of<br>
>> this message, you must delete it immediately and notify the sender. Any<br>
>> unauthorized use or disclosure of this message is strictly prohibited.<br>
>> Faurecia does not guarantee the integrity of this transmission and shall<br>
>> therefore never be liable if the message is altered or falsified nor for any<br>
>> virus, interception or damage to your system.<br>
>><br>
>> _______________________________________________<br>
>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>><br>
>> Visit other Kitware open-source projects at<br>
>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>><br>
>> Follow this link to subscribe/unsubscribe:<br>
>> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>><br>
><br>
> <a href="http://www.paraview.org/Wiki/VTK/Git#Hooks" target="_blank">http://www.paraview.org/Wiki/VTK/Git#Hooks</a> Will show you how to copy the<br>
> server commit checks for use on local commits.<br>
> --<br>
> Robert Maynard<br>
><br>
> _______________________________________________<br>
> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
><br>
> Visit other Kitware open-source projects at<br>
> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
><br>
> Follow this link to subscribe/unsubscribe:<br>
> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
><br>
><br>
><br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br>