Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 5 Feb 2023 15:51:45 +0800
From:      Po-Chuan Hsieh <sunpoet@freebsd.org>
To:        Muhammad Moinur Rahman <bofh@freebsd.org>
Cc:        ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org,  dev-commits-ports-main@freebsd.org
Subject:   Re: git: 9ba756b0e950 - main - graphics/gdal-grass: Fix PORTVERSION going backwords
Message-ID:  <CAMHz58QH1zr0A1DqAwq8a2dC1zd9y-8vrcTcs-2tnR89RyxrYA@mail.gmail.com>
In-Reply-To: <202301311256.30VCus3M012261@gitrepo.freebsd.org>
References:  <202301311256.30VCus3M012261@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000d3efab05f3ef31c0
Content-Type: text/plain; charset="UTF-8"

On Tue, Jan 31, 2023 at 8:56 PM Muhammad Moinur Rahman <bofh@freebsd.org>
wrote:

> The branch main has been updated by bofh:
>
> URL:
> https://cgit.FreeBSD.org/ports/commit/?id=9ba756b0e950d76c63a291797407abd9972d1eb2
>
> commit 9ba756b0e950d76c63a291797407abd9972d1eb2
> Author:     Muhammad Moinur Rahman <bofh@FreeBSD.org>
> AuthorDate: 2023-01-31 12:55:14 +0000
> Commit:     Muhammad Moinur Rahman <bofh@FreeBSD.org>
> CommitDate: 2023-01-31 12:56:47 +0000
>
>     graphics/gdal-grass: Fix PORTVERSION going backwords
>
>     This port was resurrected with an older version than it was in the tree
>     while removal. Hence it requires PORTEPOCH.
>

Thanks for your help but the fix is incorrect.
It is unreasonable to add PORTEPOCH when the previous one was removed over
7 years ago (Nov 1, 2015) [1].
No one would be affected with this version.

The PORTVERSION check is great.
It helps people to keep ports/pkg up-to-date (from one *installed* version
to another).
IMHO, the correct fix is to clean up the version db regularly.
We could expire removed ports after a period of time.

Thanks.

[1]
https://cgit.freebsd.org/ports/commit/?id=5e1001eeb3e8c92ec3453e2cee94a870ee99ed86


>     Approved by:    portmgr (just-fix-it)
> ---
>  graphics/gdal-grass/Makefile | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/graphics/gdal-grass/Makefile b/graphics/gdal-grass/Makefile
> index cdb9ff57bb33..fc09da52da9c 100644
> --- a/graphics/gdal-grass/Makefile
> +++ b/graphics/gdal-grass/Makefile
> @@ -1,5 +1,6 @@
>  PORTNAME=      gdal-grass
>  PORTVERSION=   1.0.2
> +PORTEPOCH=     1
>  CATEGORIES=    graphics
>
>  MAINTAINER=    sunpoet@FreeBSD.org
>
>

--000000000000d3efab05f3ef31c0
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">On Tue, Jan 31, 2023 at 8:56 PM Muhammad =
Moinur Rahman &lt;<a href=3D"mailto:bofh@freebsd.org">bofh@freebsd.org</a>&=
gt; wrote:<br></div><div class=3D"gmail_quote"><blockquote class=3D"gmail_q=
uote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,2=
04);padding-left:1ex">The branch main has been updated by bofh:<br>
<br>
URL: <a href=3D"https://cgit.FreeBSD.org/ports/commit/?id=3D9ba756b0e950d76=
c63a291797407abd9972d1eb2" rel=3D"noreferrer" target=3D"_blank">https://cgi=
t.FreeBSD.org/ports/commit/?id=3D9ba756b0e950d76c63a291797407abd9972d1eb2</=
a><br>
<br>
commit 9ba756b0e950d76c63a291797407abd9972d1eb2<br>
Author:=C2=A0 =C2=A0 =C2=A0Muhammad Moinur Rahman &lt;bofh@FreeBSD.org&gt;<=
br>
AuthorDate: 2023-01-31 12:55:14 +0000<br>
Commit:=C2=A0 =C2=A0 =C2=A0Muhammad Moinur Rahman &lt;bofh@FreeBSD.org&gt;<=
br>
CommitDate: 2023-01-31 12:56:47 +0000<br>
<br>
=C2=A0 =C2=A0 graphics/gdal-grass: Fix PORTVERSION going backwords<br>
<br>
=C2=A0 =C2=A0 This port was resurrected with an older version than it was i=
n the tree<br>
=C2=A0 =C2=A0 while removal. Hence it requires PORTEPOCH.<br></blockquote><=
div><br></div><div>Thanks for your help but the fix is incorrect.</div><div=
>It is unreasonable to add PORTEPOCH=C2=A0when the previous one was removed=
 over 7 years ago (Nov 1, 2015) [1].</div><div>No one would be affected wit=
h this version.</div><div><br></div><div>The PORTVERSION check is great.</d=
iv><div>It helps people to keep ports/pkg up-to-date (from one *installed* =
version to another).</div><div></div><div>IMHO, the correct fix is to clean=
 up the version db regularly.</div><div>We could expire removed ports after=
 a period of time.</div><div><br></div><div>Thanks.</div><div><br></div><di=
v>[1]=C2=A0<a href=3D"https://cgit.freebsd.org/ports/commit/?id=3D5e1001eeb=
3e8c92ec3453e2cee94a870ee99ed86">https://cgit.freebsd.org/ports/commit/?id=
=3D5e1001eeb3e8c92ec3453e2cee94a870ee99ed86</a></div><div><br></div><blockq=
uote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1p=
x solid rgb(204,204,204);padding-left:1ex">
<br>
=C2=A0 =C2=A0 Approved by:=C2=A0 =C2=A0 portmgr (just-fix-it)<br>
---<br>
=C2=A0graphics/gdal-grass/Makefile | 1 +<br>
=C2=A01 file changed, 1 insertion(+)<br>
<br>
diff --git a/graphics/gdal-grass/Makefile b/graphics/gdal-grass/Makefile<br=
>
index cdb9ff57bb33..fc09da52da9c 100644<br>
--- a/graphics/gdal-grass/Makefile<br>
+++ b/graphics/gdal-grass/Makefile<br>
@@ -1,5 +1,6 @@<br>
=C2=A0PORTNAME=3D=C2=A0 =C2=A0 =C2=A0 gdal-grass<br>
=C2=A0PORTVERSION=3D=C2=A0 =C2=A01.0.2<br>
+PORTEPOCH=3D=C2=A0 =C2=A0 =C2=A01<br>
=C2=A0CATEGORIES=3D=C2=A0 =C2=A0 graphics<br>
<br>
=C2=A0MAINTAINER=3D=C2=A0 =C2=A0 sunpoet@FreeBSD.org<br>
<br>
</blockquote></div></div>

--000000000000d3efab05f3ef31c0--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAMHz58QH1zr0A1DqAwq8a2dC1zd9y-8vrcTcs-2tnR89RyxrYA>