<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Aug 22, 2013, at 12:46 AM, Carlos Garcia Campos <<a href="mailto:carlosgc@webkit.org">carlosgc@webkit.org</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">El jue, 22-08-2013 a las 00:41 -0700, Mark Rowe escribió:<br><blockquote type="cite">On 2013-08-21, at 11:11 PM, Carlos Garcia Campos <<a href="mailto:carlosgc@webkit.org">carlosgc@webkit.org</a>> wrote:<br><br><blockquote type="cite">I see, I thought ChangeLog parser was used everywhere. So, I guess the<br>solution would be to find a format most people like and adapt all<br>scripts to it. I personally think it's not worth it, though. The oneline<br>git log format is ok with the old commit message format IMO. If the only<br>problem is that the bug number is not in the first line we can probably<br>add it without adding the URL in angle brackets, something like:<br><br>Bug 119872 - REGRESSION: Crash under JITCompiler::link while loading Gmail<br><a href="https://bugs.webkit.org/show_bug.cgi?id=119872">https://bugs.webkit.org/show_bug.cgi?id=119872</a><br></blockquote><br>What benefit does duplicating the bug number in the first line provide? It’s just yet another thing to have to add when writing the ChangeLog entry.<br></blockquote><br>The benefit is that you see the bug number when using git log oneline<br>format, I'm guessing that seeing the bug number in the first line was<br>the motivation of the new ChangeLog format.<br></blockquote></div><div><br></div><div>I prefer the new format. However, the new format doesn't really gain us much, so it sounds like it is not worth the effort to change.</div><div><br></div><div>As for duplicating the bug number, I don't think that matters. `git log --oneline` shows a short SHA the "title line" which is everything in the commit message until the first empty line. So duplicating the bug number in the first line doesn't really help, since both of these lines already show up.</div><div><br></div><div>Old format:</div><div><br></div><div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;"><div>shell> git log --oneline `git svn find-rev r153963` -10</div></blockquote></div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;"><div><div><div><div>503293b Move additionalIdentifiers into DFGCommonData as only the optimising JITs use them <a href="https://bugs.webkit.org/show_bug.cgi?id=119705">https://bugs.webkit.org/show_bug.cgi?id=119705</a></div><div>2dbc921 Stop making unnecessary copy of CodeBlock Identifier Vector <a href="https://bugs.webkit.org/show_bug.cgi?id=119702">https://bugs.webkit.org/show_bug.cgi?id=119702</a></div><div>c0c68df [cairo] memory corruption with putImageData and accelerated canvas. <a href="https://bugs.webkit.org/show_bug.cgi?id=118621">https://bugs.webkit.org/show_bug.cgi?id=118621</a></div><div>0502819 Layout Test fast/overflow/scrollbar-click-retains-focus.html is failing on Apple Mac WK1 and GTK. <a href="https://bugs.webkit.org/show_bug.cgi?id=104104">https://bugs.webkit.org/show_bug.cgi?id=104104</a></div><div>c19a87b Use const AtomicStringImpl* in DocumentOrderedMap <a href="https://bugs.webkit.org/show_bug.cgi?id=119700">https://bugs.webkit.org/show_bug.cgi?id=119700</a></div><div>02c59d8 The LayoutMilestones didLayout callback should only fire for the main frame <a href="https://bugs.webkit.org/show_bug.cgi?id=119457">https://bugs.webkit.org/show_bug.cgi?id=119457</a></div><div>2cb3317 Remove didNewFirstVisuallyNonEmptyLayout from WebKit2 <a href="https://bugs.webkit.org/show_bug.cgi?id=119658">https://bugs.webkit.org/show_bug.cgi?id=119658</a></div><div>6cb5165 Define DOM_KEY_LOCATION_* constants on KeyboardEvent <a href="https://bugs.webkit.org/show_bug.cgi?id=119341">https://bugs.webkit.org/show_bug.cgi?id=119341</a></div><div>597845d Remove glu's libtest rule from checker.py <a href="https://bugs.webkit.org/show_bug.cgi?id=119674">https://bugs.webkit.org/show_bug.cgi?id=119674</a></div><div>35852cd Multiple CSP tests complain about undefined strings in echo-object-data.pl and echo-script-src.pl <a href="https://bugs.webkit.org/show_bug.cgi?id=119692">https://bugs.webkit.org/show_bug.cgi?id=119692</a></div></div></div></div><div><br></div></blockquote><div>New format:</div><div><br></div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;"><div>shell> git log --oneline `git svn find-rev r154274` -10</div><div><div><div>c3bd590 <<a href="https://webkit.org/b/119882">https://webkit.org/b/119882</a>> Speed up moving cursor/selection up or down past non-rendered elements.</div><div>6c326ac <<a href="https://webkit.org/b/119933">https://webkit.org/b/119933</a>> [Qt] Fix tst_QObjectBridge::connectAndDisconnect() fail</div><div>381c093 <<a href="https://webkit.org/b/120000">https://webkit.org/b/120000</a>> Chrome::client() should return a reference.</div><div>f183c4b <<a href="https://webkit.org/b/120004">https://webkit.org/b/120004</a>> Rename StyleElement to InlineStyleSheetOwner and stop inheriting from it</div><div>0d74bc3 [Qt] QtWebKit (using the Arora browser) displays the border radii (radius) of a button very ugly <a href="https://bugs.webkit.org/show_bug.cgi?id=28113">https://bugs.webkit.org/show_bug.cgi?id=28113</a></div><div>e5a4a31 <<a href="https://webkit.org/b/119998">https://webkit.org/b/119998</a>> [Qt] Build fix (broken since r154257).</div><div>d3ac166 <<a href="https://webkit.org/b/120001">https://webkit.org/b/120001</a>> Clean up StyleElement</div><div>5c403bd <<a href="https://webkit.org/b/119990">https://webkit.org/b/119990</a>> Add toSVGStopElement(Node* node) to clean-up a static_cast<SVGStopElement*></div><div>508cc36 <<a href="https://webkit.org/b/119996">https://webkit.org/b/119996</a>> Introduce toSVGAnimateElement(), and use it</div><div>db5024b <<a href="https://webkit.org/b/119997">https://webkit.org/b/119997</a>> Page::progress() should return a reference.</div></div></div></blockquote><br>- Joe</body></html>