<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Mar 20, 2022, 4:58 PM Ruben Schuller <<a href="mailto:sb@rbn.im">sb@rbn.im</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
2022-03-20 "B. Watson" <<a href="mailto:urchlay@slackware.uk" target="_blank" rel="noreferrer">urchlay@slackware.uk</a>>:<br>
<br>
> On Sat, 19 Mar 2022, Ruben Schuller wrote:<br>
> ><br>
> > ZIPVER="1369d247e3d2bc697892795dea8cc1713344749f"<br>
> ><br>
> > for the version I'd either go with the date like 220319 if there<br>
> > aren't multiple commits on that date, or with the short-hash of<br>
> > git:  <br>
> <br>
> Recommendations... in fact I'm begging you: if you're going to use<br>
> a git commit, *please* include the date in YYMMDD format in your<br>
> VERSION number. Example:<br>
> <br>
> VERSION="20220319+1369d24"<br>
> <br>
> Notice the 4-digit year comes first... the point of doing it this way<br>
> is that a user can easily tell which version is newer, when looking<br>
> at multiple packages. If you use only the git commit hash, those are<br>
> effectively random, there's no sense of which is newest.<br>
> <br>
> The SlackBuild can split up the VERSION variable as needed.<br>
> <br>
> Also, while I'm thinking about it: VERSION *must* be a constant that<br>
> can be overridden by the environment. Example:<br>
> <br>
> VERSION=${VERSION:-3.4.1}<br>
<br>
Very good point, including the date never hurts.<br>
<br>
> *Please* don't try to get clever and do stuff like:<br>
> <br>
> COMMIT=1369d247e3d2bc697892795dea8cc1713344749f<br>
> VERSION="$( echo $COMMIT | head -c7 )"<br>
<br>
Note, for completeness, that the git short hash might be longer than<br>
seven characters because of collisions. The git command figures that<br>
out. Don't just shorten it :)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">On SBo, it won't matter if there's collisions. It's simply a reference to a version that exists on SBo. If there's collisions within the upstream repo, it won't affect the version we have on SBo.</div><div dir="auto"><br></div><div dir="auto">Jeremy</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">
</blockquote></div></div></div>