<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >Hey Stewart,</div>
<div dir="ltr" > </div>
<div dir="ltr" >Thanks again for your information.</div>
<div dir="ltr" > </div>
<div dir="ltr" >I have updated the message of my commits in my patch, please review that and let me know if the issue still exists.</div>
<div dir="ltr" > </div>
<div dir="ltr" >Please note that when I use git rebase to edit the message, I take 1 more commit into my pool. I actually changed nothing for that commit, but it appears in pull request. Please ignore that commit in my pull request.</div>
<div dir="ltr" > </div>
<div dir="ltr" ><img src="cid:14502333509510" ></div>
<div dir="ltr" > </div>
<div dir="ltr" >GOU, Peng Fei (苟鹏飞), Ph.D.</div>
<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >OpenPower Team.</div>
<div dir="ltr" >+86-21-609-28631</div></div></div>
<div dir="ltr" > </div>
<div dir="ltr" > </div>
<blockquote data-history-content-modified="1" dir="ltr" style="border-left:solid #aaaaaa 2px; margin-left:5px; padding-left:5px; direction:ltr; margin-right:0px" >----- Original message -----<br>From: Stewart Smith <stewart@linux.vnet.ibm.com><br>To: Peng Fei BG Gou/China/IBM@IBMCN<br>Cc: openbmc-patches@stwcx.xyz, openbmc@lists.ozlabs.org<br>Subject: Re: [PATCH phosphor-host-ipmid v2 1/5] Add get/set ipmid command support with correct DBUS property handling.<br>Date: Wed, Dec 16, 2015 7:29 PM<br> 
<div><font face="Default Monospace,Courier New,Courier,monospace" size="2" >Peng Fei BG Gou <shgoupf@cn.ibm.com> writes:<br>> I'm sorry that for my commits I only have "1 line" comments for them.<br>> I will try to fix them by changing the comments that already been<br>> committed. But there are plenty of commits that is "1 line" comments<br>> as below. So would you mind telling me in what situation I am allowed<br>> to have "1 line" comment and in what situation I need to have a<br>> multiple-line comment?<br><br>Yeah, there's a lot of poor commit messages in OpenBMC so far, which<br>gives you the opportunity to be much better than others!<br><br>Personally, my guide is: if someone casually familiar with the project<br>and concepts can see that the patch is obviously correct in less than<br>four seconds, a one line commit message is okay.<br><br>My goal for a commit message is to give enough information to someone<br>casually familiar with the project and subject matter enough to go and<br>start to learn enough to completely understand the patch.<br><br>Even then, sometimes a bit more info is useful.<br>For example:<br><a href="https://github.com/open-power/skiboot/commit/d4b142823e28deed097cf79b25d8d1e60b6f4e67" target="_blank" >https://github.com/open-power/skiboot/commit/d4b142823e28deed097cf79b25d8d1e60b6f4e67</a><br><br>An example of single line commit message:<br><a href="https://github.com/open-power/skiboot/commit/916196d4cc0e070369ff6aa1363fa39339f8cad1" target="_blank" >https://github.com/open-power/skiboot/commit/916196d4cc0e070369ff6aa1363fa39339f8cad1</a><br>(which is just adding release notes, so it's *obvious* what this patch<br>does)<br><br>Or<br><a href="https://github.com/torvalds/linux/commit/5c477b45792fb8ccd279162483aa57cc8ee40ae1" target="_blank" >https://github.com/torvalds/linux/commit/5c477b45792fb8ccd279162483aa57cc8ee40ae1</a><br>which is just the wiring up of a new syscall for PARISC.</font></div></blockquote>
<div dir="ltr" > </div></div>
<BR>