<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">Le jeu. 25 oct. 2018 à 00:50, Fedor Sergeev via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On 10/25/18 1:09 AM, Renato Golin via llvm-dev wrote:<br>
> On Wed, 24 Oct 2018 at 22:45, NAKAMURA Takumi via llvm-dev<br>
> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>> Let me speak bikeshedding.<br>
>> I was thinking what is better to represent svn revision number in each commit.<br>
>><br>
>> I propose just to add svnrev in front of subject, instead of appending metadata.<br>
>> "rXXXXXX: This is 1st line of commit log"<br>
> That's a good question, and the answer will define how we find svn<br>
> revs in the future. But I don't think there are a lot of wrong answers<br>
> here, we just have to pick one and stick to it.<br>
><br>
> I don't know enough about how git queries commit messages to have a<br>
> strong opinion, but I imagine on subject is easier to grab than the<br>
> rest of the message, and it will show in GitHub's commit list.<br>
For the purpose of just searching through the log it does not matter.<br>
git log --grep= looks through the whole commit message.<br>
<br>
Right now svn rev number appears to be at the bottom of commit message,<br>
which works fine for git log investigation purposes.<br>
> It would be worse if we had hard rules against long subjects, but we<br>
> don't, so I guess it's ok from that point of view.<br>
><br>
> We normally use [TAG], so [rXXX] would "look" more natural to me, but<br>
> that's beyond bikeshedding, so whatever works. :)<br>
We seem to use TAG things to put "semantical" markers into the subject <br>
line to be able<br>
to quickly sort out interesting topics. Exactly because of that I do dislike<br>
suggestion to put svn rev numbers into the subject, since svn rev number<br>
does not have any semantical meaning. [rXXX] would not be any different for<br>
me than [rYYY] so why bother putting it there at all?<br></blockquote><div><br></div><div>Same opinion for me: the first line of a commit in git is per convention usually a short "title" that should describe what a patch does.</div><div>An SVN revision number isn't the main thing I'm looking for when looking for a patch.</div><div>In the future (post-move), SVN revision will become less and less useful and only important for history/archeology purpose: this does not seem like it justifies "let put it in front of everything" as if it is always gonna be the most important thing about a particular commit.</div><div><br></div><div>-- </div><div>Mehdi</div></div></div>