Date: Wed, 23 Aug 2023 00:57:51 +0800 From: Po-Chuan Hsieh <sunpoet@freebsd.org> To: rodrigo@freebsd.org Cc: Joseph Mingrone <jrm@freebsd.org>, Renato Botelho <garga@freebsd.org>, ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: 11e51925413b - main - devel/p5-DateTime-Format-MySQL: Update to 0.08 Message-ID: <CAMHz58RdUg-FBhFgB%2BaSu2=MG=9fA7A6zoCgB%2B4qpAs12oFdFw@mail.gmail.com> In-Reply-To: <226220f0-119d-79d1-c597-064990d6cb6f@FreeBSD.org> References: <202308161836.37GIaHp6043137@gitrepo.freebsd.org> <ec2e809f-9b65-46cb-b4fc-cacd287c5a83@FreeBSD.org> <86jzttx00m.fsf@phe.ftfl.ca> <a1c25f10-8198-4c67-891d-aa6a9ce5ae02@FreeBSD.org> <867cpsp4hi.fsf@phe.ftfl.ca> <6cb25b46-33e1-4ab6-8207-c3087c08a7a5@FreeBSD.org> <867cpsnmuh.fsf@phe.ftfl.ca> <CAMHz58TqGh9pO3gFsAmqth7tP4NSBsxZ7iJQk2ynCuJq-j-e7w@mail.gmail.com> <226220f0-119d-79d1-c597-064990d6cb6f@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000006c4792060385e77a Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Wed, Aug 23, 2023 at 12:19=E2=80=AFAM Rodrigo Osorio <rodrigo@freebsd.or= g> wrote: > On 21/08/23 20:11, Po-Chuan Hsieh wrote: > > On Sat, Aug 19, 2023 at 3:59=E2=80=AFAM Joseph Mingrone <jrm@freebsd.org>= wrote: > >> On Fri, 2023-08-18 at 16:08, Renato Botelho <garga@FreeBSD.org> >> <garga@FreeBSD.org> wrote: >> >> > On 18/08/23 15:52, Joseph Mingrone wrote: >> >> On Fri, 2023-08-18 at 14:32, Renato Botelho <garga@FreeBSD.org> >> <garga@FreeBSD.org> wrote: >> >> >>> On 17/08/23 10:37, Joseph Mingrone wrote: >> >>>> On Thu, 2023-08-17 at 10:15, Renato Botelho <garga@FreeBSD.org> >> <garga@FreeBSD.org> wrote: >> >> >>>>> On 16/08/23 15:36, Po-Chuan Hsieh wrote: >> >>>>>> The branch main has been updated by sunpoet: >> >>>>>> URL: >> https://cgit.FreeBSD.org/ports/commit/?id=3D11e51925413b7090544d4a4ac449= c364717f7e50 >> >>>>>> commit 11e51925413b7090544d4a4ac449c364717f7e50 >> >>>>>> Author: Po-Chuan Hsieh <sunpoet@FreeBSD.org> >> <sunpoet@FreeBSD.org> >> >>>>>> AuthorDate: 2023-08-16 18:16:29 +0000 >> >>>>>> Commit: Po-Chuan Hsieh <sunpoet@FreeBSD.org> >> <sunpoet@FreeBSD.org> >> >>>>>> CommitDate: 2023-08-16 18:26:21 +0000 >> >>>>>> devel/p5-DateTime-Format-MySQL: Update to 0.08 >> >>>>>> Changes: >> https://metacpan.org/dist/DateTime-Format-MySQL/changes >> >>>>>> --- >> >>>>>> devel/p5-DateTime-Format-MySQL/Makefile | 2 +- >> >>>>>> devel/p5-DateTime-Format-MySQL/distinfo | 6 +++--- >> >>>>>> 2 files changed, 4 insertions(+), 4 deletions(-) >> >>>>>> diff --git a/devel/p5-DateTime-Format-MySQL/Makefile >> b/devel/p5-DateTime-Format-MySQL/Makefile >> >>>>>> index 989d481a74d2..f9b0e03be278 100644 >> >>>>>> --- a/devel/p5-DateTime-Format-MySQL/Makefile >> >>>>>> +++ b/devel/p5-DateTime-Format-MySQL/Makefile >> >>>>>> @@ -1,5 +1,5 @@ >> >>>>>> PORTNAME=3D DateTime-Format-MySQL >> >>>>>> -PORTVERSION=3D 0.0701 >> >>>>>> +PORTVERSION=3D 0.08 >> >> >>>>> Hello Po-Chuan! >> >> >>>>> 0.08 is smaller than 0.0701 so port version went backwards after >> this change. >> >>>> 0.08 is greater than 0.0701, but not according to pkg version. >> >>>> % pkg version -t 0.08 0.0701 >> >>>> < >> >> >>> Actually pkg is correct in this case. Numerically 8 is smaller than >> 701. I tested it on Ubuntu and dpkg --compare-versions has the same >> result, and I believe any other package management tool will do. >> >> >>> What seems to be wrong is the version schema used by this project. >> It should be 0.0800 instead of 0.08. >> >> I didn't mean to suggest that pkg is wrong, but just pointing out tha= t >> numerically, 0.08, which is equal to 0.0800, is greater than 0.0701. Th= is >> could be a source of confusion for anyone who assumes that the compariso= n >> of version numbers works the same as comparisons of >> >> real numbers. >> >> > Anyway, can you please fix the port by adding PORTEPOCH? >> >> Fixed in 49eb4309189da197f55dee54f8efb5e875f2daaa. Thanks for reporting= . >> >> Joe >> > > No doubt 0.08 is less than 0.0701 in the pkg world. > Thank you all for fixing it. > > We don't know if the versioning change is temporary or not. Instead of > adding PORTEPOCH, please consider the following fix/hack next time. > Thanks again. > > --- a/devel/p5-DateTime-Format-MySQL/Makefile > +++ b/devel/p5-DateTime-Format-MySQL/Makefile > @@ -1,8 +1,9 @@ > PORTNAME=3D DateTime-Format-MySQL > -PORTVERSION=3D 0.08 > +PORTVERSION=3D 0.0800 > CATEGORIES=3D devel perl5 > MASTER_SITES=3D CPAN > PKGNAMEPREFIX=3D p5- > +DISTNAME=3D DateTime-Format-MySQL-${PORTVERSION:C|00$||} > > MAINTAINER=3D perl@FreeBSD.org > COMMENT=3D Parse and format MySQL dates and times > > > Hi, > > I don't know if such a simple fix can solve the issue we have with Perl > versioning sheme > and the semantic versioning scheme we use in ports. > This article is very instructive about how awful it is > https://perlhacks.com/2016/12/version-numbers/. > Yes, Perl has a different versioning scheme. That's why I would hack PORTVERSION sometimes. Take textproc/p5-Text-ANSI-Util for example, I removed the hack after years since the version scheme seems to be consolidated. c975054b7866b05594a1b6be9cea3c33870d8567 11cb372ff4cae86977fe2f873b630825b8cd43a0 Just a few examples running this command to evaluate the version number: > perl -Mversion -E"say version->parse('$VERSION')->normal" > > VERSION | Output > -----------|------- > "8" | v8.0.0 > "00008" | v8.0.0 > "000080" | v80.0.0 > "0.8" | v0.800.0 > "0.81" | v0.810.0 > "0.801" | v0.801.0 > "0.8001" | v0.800.100 > "0.008001" | v0.8.1 > > By the way, I did a quick try in with your fix, and the substitution with > an '$' > doesn't seem to work in BSD makefiles. > > %cat Makefile > V=3D"0.0800" > You have to remove the double quote. > all: > @echo ${V} > @echo ${V:C|00$||} > > %make > 0.0800 > 0.0800 > > Regards > -- rodrigo > --0000000000006c4792060385e77a Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr">On Wed, Aug 23, 2023 at 12:19=E2=80=AFAM = Rodrigo Osorio <<a href=3D"mailto:rodrigo@freebsd.org">rodrigo@freebsd.o= rg</a>> wrote:<br></div><div class=3D"gmail_quote"><blockquote class=3D"= gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(20= 4,204,204);padding-left:1ex"> =20 =20 =20 <div> <div>On 21/08/23 20:11, Po-Chuan Hsieh wrote:<br> </div> <blockquote type=3D"cite"> =20 <div dir=3D"ltr"> <div dir=3D"ltr">On Sat, Aug 19, 2023 at 3:59=E2=80=AFAM Joseph Min= grone <<a href=3D"mailto:jrm@freebsd.org" target=3D"_blank">jrm@free= bsd.org</a>> wrote:<br> </div> <div class=3D"gmail_quote"> <blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8= ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, 2023-08= -18 at 16:08, Renato Botelho <a href=3D"mailto:garga@FreeBSD.org" targ= et=3D"_blank"><garga@FreeBSD.org></a> wrote:<br> <br> > On 18/08/23 15:52, Joseph Mingrone wrote:<br> >> On Fri, 2023-08-18 at 14:32, Renato Botelho <a href=3D"mailto:garga@FreeBSD.org" target=3D"_blank"><garg= a@FreeBSD.org></a> wrote:<br> <br> >>> On 17/08/23 10:37, Joseph Mingrone wrote:<br> >>>> On Thu, 2023-08-17 at 10:15, Renato Botelho <a href=3D"mailto:garga@FreeBSD.org" target=3D"_blank"><garg= a@FreeBSD.org></a> wrote:<br> <br> >>>>> On 16/08/23 15:36, Po-Chuan Hsieh wrote:<br> >>>>>> The branch main has been updated by sunpoet:<br> >>>>>> URL: <a href=3D"https://cgit.FreeBSD.o= rg/ports/commit/?id=3D11e51925413b7090544d4a4ac449c364717f7e50" rel=3D"nore= ferrer" target=3D"_blank">https://cgit.FreeBSD.org/ports/commit/?id=3D11e51= 925413b7090544d4a4ac449c364717f7e50</a><br> >>>>>> commit 11e51925413b7090544d4a4ac449c364717f7e50<br> >>>>>> Author:=C2=A0 =C2=A0 =C2=A0Po-Chuan Hs= ieh <a href=3D"mailto:sunpoet@FreeBSD.org" target=3D"_blank"><su= npoet@FreeBSD.org></a><br> >>>>>> AuthorDate: 2023-08-16 18:16:29 +0000<br> >>>>>> Commit:=C2=A0 =C2=A0 =C2=A0Po-Chuan Hs= ieh <a href=3D"mailto:sunpoet@FreeBSD.org" target=3D"_blank"><su= npoet@FreeBSD.org></a><br> >>>>>> CommitDate: 2023-08-16 18:26:21 +0000<br> >>>>>>=C2=A0 =C2=A0 =C2=A0 =C2=A0 devel/p5-DateTime-Format-MySQL: Update to 0.08<br> >>>>>>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0Changes:=C2=A0 =C2=A0 =C2=A0 =C2=A0 <a href=3D"https://metacpan.o= rg/dist/DateTime-Format-MySQL/changes" rel=3D"noreferrer" target=3D"_blank"= >https://metacpan.org/dist/DateTime-Format-MySQL/changes</a><br>; >>>>>> ---<br> >>>>>>=C2=A0 =C2=A0 =C2=A0devel/p5-DateTime-Format-MySQL/Makefile | 2 +-<br> >>>>>>=C2=A0 =C2=A0 =C2=A0devel/p5-DateTime-Format-MySQL/distinfo | 6 +++---<br> >>>>>>=C2=A0 =C2=A0 =C2=A02 files changed, 4 insertions(+), 4 deletions(-)<br> >>>>>> diff --git a/devel/p5-DateTime-Format-MySQL/Makefile b/devel/p5-DateTime-Format-MySQL/Makefile<br> >>>>>> index 989d481a74d2..f9b0e03be278 100644<br> >>>>>> --- a/devel/p5-DateTime-Format-MySQL/Makefile<br> >>>>>> +++ b/devel/p5-DateTime-Format-MySQL/Makefile<br> >>>>>> @@ -1,5 +1,5 @@<br> >>>>>>=C2=A0 =C2=A0 =C2=A0PORTNAME=3D=C2=A0 = =C2=A0 DateTime-Format-MySQL<br> >>>>>> -PORTVERSION=3D=C2=A0 =C2=A0 0.0701<br= > >>>>>> +PORTVERSION=3D=C2=A0 =C2=A0 0.08<br> <br> >>>>> Hello Po-Chuan!<br> <br> >>>>> 0.08 is smaller than 0.0701 so port version went backwards after this change.<br> >>>> 0.08 is greater than 0.0701, but not according to pkg version.<br> >>>>=C2=A0 =C2=A0 % pkg version -t 0.08 0.0701<br> >>>>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0<<b= r> <br> >>> Actually pkg is correct in this case.=C2=A0 Numerically 8 is smaller than 701.=C2=A0 I tested it on Ubuntu and dpkg --compare-versions has the same result, and I believe any other package management tool will do.<br> <br> >>> What seems to be wrong is the version schema used by this project.=C2=A0 It should be 0.0800 instead of 0.08= .<br> >> I didn't mean to suggest that pkg is wrong, but just pointing out that numerically, 0.08, which is equal to 0.0800, is greater than 0.0701.=C2=A0 This could be a source of confusion for anyone who assumes that the comparison of version numbers works the same as comparisons of<br> >> real numbers.<br> <br> > Anyway, can you please fix the port by adding PORTEPOCH?<br> <br> Fixed in 49eb4309189da197f55dee54f8efb5e875f2daaa.=C2=A0 Thanks for reporting.<br> <br> Joe<br> </blockquote> <div><br> </div> <div>No doubt 0.08 is less than 0.0701 in the pkg world.</div> <div>Thank you all for fixing it.<br> </div> <div><br> </div> <div>We don't know if the versioning change is temporary or not. Instead of adding PORTEPOCH, please consider the following fix/hack next time.<br> </div> <div>Thanks again.</div> <div><br> </div> --- a/devel/p5-DateTime-Format-MySQL/Makefile<br> +++ b/devel/p5-DateTime-Format-MySQL/Makefile<br> @@ -1,8 +1,9 @@<br> =C2=A0PORTNAME=3D =C2=A0 =C2=A0 =C2=A0DateTime-Format-MySQL<br> -PORTVERSION=3D =C2=A0 0.08<br> +PORTVERSION=3D =C2=A0 0.0800<br> =C2=A0CATEGORIES=3D =C2=A0 =C2=A0devel perl5<br> =C2=A0MASTER_SITES=3D =C2=A0CPAN<br> =C2=A0PKGNAMEPREFIX=3D p5-<br> +DISTNAME=3D =C2=A0 =C2=A0 =C2=A0DateTime-Format-MySQL-${PORTVERS= ION:C|00$||}<br> <br> =C2=A0MAINTAINER=3D =C2=A0 =C2=A0<a href=3D"mailto:perl@FreeBSD.o= rg" target=3D"_blank">perl@FreeBSD.org</a><br> <div>=C2=A0COMMENT=3D =C2=A0 =C2=A0 =C2=A0 Parse and format MySQL= dates and times</div> <div>=C2=A0</div> </div> </div> </blockquote> <p>Hi,</p> <p>I don't know if such a simple fix can solve the issue we have with Perl versioning sheme<br> and the semantic versioning scheme we use in ports.<br> This article is very instructive about how awful it is <a href=3D"https://perlhacks.com/2016/12/version-numbers/" target=3D"= _blank">https://perlhacks.com/2016/12/version-numbers/</a>.</p></div></bloc= kquote><div><br></div><div>Yes, Perl has a different versioning scheme.</di= v><div>That's why I would hack PORTVERSION sometimes.</div><div>Take te= xtproc/p5-Text-ANSI-Util for example, I removed the hack after years since = the version scheme seems to be consolidated.<br></div><div>c975054b7866b055= 94a1b6be9cea3c33870d8567</div><div>11cb372ff4cae86977fe2f873b630825b8cd43a0= </div><div><br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px = 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div= > <p>Just a few examples running this command to evaluate the version number:<br> =C2=A0=C2=A0=C2=A0 perl -Mversion -E"say version->parse('$VERSION')->normal"<br> <br> VERSION=C2=A0=C2=A0=C2=A0 | Output<br> -----------|-------<br> "8"=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 | v8.0.0<br> "00008"=C2=A0=C2=A0=C2=A0 | v8.0.0<br> "000080"=C2=A0=C2=A0 | v80.0.0<br> "0.8"=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 | v0.800.0<br> "0.81"=C2=A0=C2=A0=C2=A0=C2=A0 | v0.810.0<br> "0.801"=C2=A0=C2=A0=C2=A0 | v0.801.0 <br> "0.8001"=C2=A0=C2=A0 | v0.800.100<br> "0.008001" | v0.8.1</p> <p>By the way, I did a quick try in with your fix, and the substitution with an '$'<br> doesn't seem to work in BSD makefiles.</p> <p>%cat Makefile<br> V=3D"0.0800"</p></div></blockquote><div>You have to remove = the double quote.</div><blockquote class=3D"gmail_quote" style=3D"margin:0p= x 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><d= iv><p> all:<br> =C2=A0=C2=A0=C2=A0 @echo ${V}<br> =C2=A0=C2=A0=C2=A0 @echo ${V:C|00$||}</p> <p>%make<br> 0.0800<br> 0.0800</p> <p>Regards<br> -- rodrigo<br> </p> </div> </blockquote></div></div> --0000000000006c4792060385e77a--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAMHz58RdUg-FBhFgB%2BaSu2=MG=9fA7A6zoCgB%2B4qpAs12oFdFw>