<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 19, 2018, 13:21 David Greene 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">[ Consider all of this to be preferences, not anything close to an<br>
  objection. ]<br>
<br>
Tom Stellard via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank" rel="noreferrer">llvm-dev@lists.llvm.org</a>> writes:<br>
<br>
> There haven't been many more responses in the last few days, so can we<br>
> try to come to some kind of consensus here?<br>
<br>
I agree with others that this discussion doesn't have to delay the<br>
official repo from being posted.  Let's get that done ASAP.  Lots of<br>
people are waiting on it.<br>
<br>
> 1. Release tags.  There were a lot of small variation on the tag names for releases,<br>
> but it seems like the preferences was to use the <a href="http://llvm.org" rel="noreferrer noreferrer" target="_blank">llvm.org</a> prefix,<br>
> so I'm going to propose using tag names like:<br>
><br>
> llvm.org-8.0.0<br>
> llvm.org-8.0.0-rc1<br>
><br>
> Any strong objections to this?<br>
<br>
SGTM.  I don't think we need a -final suffix or anything.  A straight<br>
version number seems to convey intent for many other projects.  But not<br>
a big deal either way.<br>
<br>
We almost certainly want annotated tags.<br>
<br>
> 2. Tags for commits in the master branch that bump the release version.<br>
><br>
> Most of the discussion about this so far has been on what to put after<br>
> the version number (e.g. v8.0.0-dev, v8.0.0-base, v8.0.0-branchpoint).<br>
> Other things to consider about this tag is that it might be used in<br>
> a git describe alias to identify commits, so it would be helpful if<br>
> it was short.<br>
><br>
> One idea I had after reading through all the responses was to use the<br>
> -git suffix on the tags. e.g. v8.0.0-git.  It's short and it's clear<br>
> that you are getting something that isn't an official release.  It<br>
> also is similar to the 8.0.0svn version number that we currently use<br>
> to indicate a non-released version.  Which of these 4 options(<br>
> dev, base, branchpoint, git) do people prefer?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Of these, -git seems the least misleading, based on responses so far (branchpoint in particular). However, based on previous comments, I'm not sure we want the specific format you suggested (see previous; it's missing at least an llvm prefix namespace).</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I agree that -git or -dev could be misleading.  I also agree that we may<br>
not need this tag at all.  </blockquote></div></div><div dir="auto"><br></div><div dir="auto">Strictly speaking, we don't *need* to switch to git, either. ;-) It's worth our time collectively (I claim) to get it right from the beginning; a broken or wrong output from a built-in git command seems problematic.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If we want the tag, I thirdly agree that<br>
tagging the split point is more useful/convenient than tagging the<br>
version bump.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Tagging the split point is semantically the wrong thing, though: if you `git describe` the last commit before the branch for release ${N} (i.e., the last commit of N), the descriptor will be ${N+1} (or whatever tag was applied). The version bump commit will then be "<span style="font-family:sans-serif">${N+1}-1", meaning the second commit of the N+1 release (with commit hashes as appropriate). That's an off-by-one error in several ways. ;-)</span></div><div dir="auto"><span style="font-family:sans-serif"><br></span></div><div dir="auto"><span style="font-family:sans-serif">Also note, the split point is trivial to reach from the version bump, but not vice versa. It's reachable as ^ from the version bump tag, and also (less trivially, but still without explicit set arithmetic) as merge-base. </span></div><div dir="auto"><span style="font-family:sans-serif"><br></span></div><div dir="auto"><font face="sans-serif">An easy way to reason about this is that the version bump tag should only be reachable from the release branch for that version. So if the 8.0 bump is reachable from the 7.x release branch, that's going to result in `git describe` attributing commits with the wrong version.</font></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I have a preference for an <a href="http://llvm.org" rel="noreferrer noreferrer" target="_blank">llvm.org</a> prefix on this tag, just like for<br>
the release tags, for the same reason we have it there: namespacing to<br>
avoid downstream conflicts.  But it's certainly not a dealbreaker.<br>
<br>
                            -David<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank" rel="noreferrer">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div></div>