<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 8, 2020 at 6:57 PM Daniel Sanders via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><div>Hi All,</div><div><br></div><div>While committing a patch today for someone who currently lacks commit access, I realized I had forgotten the 'Patch by' line and reverted & re-committed to add it. However, while doing so I wondered if our current policy on this still makes sense.</div><div><br></div><div>Our policy at <a href="http://llvm.org/docs/DeveloperPolicy.html#commit-messages" target="_blank">http://llvm.org/docs/DeveloperPolicy.html#commit-messages</a> currently contains these two points:</div><div>> Separate the commit message into title, body and, if you’re not the original author, a “Patch by” attribution line (see below).</div><div>> Attribution of Changes should be in a separate line, after the end of the body, as simple as “Patch by John Doe.”. This is how we officially handle attribution, and there are automated processes that rely on this format.</div><div><br></div><div>Given that we now use git which tracks the author and the commiter separately. Does it still make sense to add these 'Patch by' lines? Or should we update the policy so that it refers to making sure the author field has the correct attribution. In my opinion, we should update the policy.</div></div></blockquote><div><br></div><div>+1 : we should change the doc to indicate to use instead `git commit --amend --author="name/nickname <email>"` .</div><div><br></div><div><br></div><div>-- </div><div>Mehdi</div><div><br></div></div></div>