<div class="gmail_quote">On Fri, Mar 9, 2012 at 6:35 PM, Pablo Flouret <span dir="ltr"><<a href="mailto:pablof@motorola.com">pablof@motorola.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


I'd prefer to see something closer to what 'git format-patch' spits out.<br>
Basically you give it a commit range and it spits out one diff per commit, including the commit message.<br></blockquote><div><br></div><div>That seems to imply we're completely changing the semantics and the format of change logs. I'd rather not get into such a discussion on this thread, and I think that's a much more controversial change.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">So you'd have a patch that includes commit messages ready for reviewers' perusal (which i presume is the biggest reason changelogs still exist?)<br>

</blockquote><div><br></div><div>Please go through the webkit-dev archive before in the past 3 years :)</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


In my most frequent workflow i make a patch, submit it, make a local commit in git for myself (in a branch) and then move on to other things (in a different branch perhaps). Then when review comments come in, i address them, make a local commit with those changes, and upload a patch by giving a range to webkit-patch. ChangeLog files are a pain in the ass in this case. Also, reviewers don't get to see what changed between the two patches i uploaded, which a patch coming from format-patch would show.</blockquote>

<div><br></div><div>Being able to see changes between two patches will be a very valuable tool. I always open two review pages side-by-side to see what has changed in new patch. However, such a review tool can be implemented on Bugzilla without moving to git.</div>

<div><br></div><div>In fact, the diff's between two local git commits is of little use for me as a reviewer because what I want to see is the diff between the latest patch and the last patch I looked at.</div><div><br>

</div><div>- Ryosuke</div><div><br></div></div>