Bug 269501 - math/py-pyclipper: Update to 1.3.0.post4
Summary: math/py-pyclipper: Update to 1.3.0.post4
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-11 20:01 UTC by Yuri Victorovich
Modified: 2023-12-11 17:55 UTC (History)
2 users (show)

See Also:
lantw44: maintainer-feedback+


Attachments
patch (1.22 KB, patch)
2023-02-11 20:01 UTC, Yuri Victorovich
no flags Details | Diff
Suggested patch (1.40 KB, patch)
2023-03-12 06:32 UTC, Ting-Wei Lan
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yuri Victorovich freebsd_committer freebsd_triage 2023-02-11 20:01:18 UTC
Created attachment 240096 [details]
patch
Comment 1 Ting-Wei Lan 2023-02-27 01:31:29 UTC
Comment on attachment 240096 [details]
patch

>--- a/math/py-pyclipper/Makefile
>+++ b/math/py-pyclipper/Makefile
>@@ -1,5 +1,5 @@
> PORTNAME=	pyclipper
>-PORTVERSION=	1.2.1
>+DISTVERSION=	1.3.0.post4

It seems the version number causes pkg to consider it a pre-release.

$ pkg version -t 1.3.0 1.3.0.p4
>
Comment 2 Ting-Wei Lan 2023-03-12 06:32:52 UTC
Created attachment 240773 [details]
Suggested patch
Comment 3 Po-Chuan Hsieh freebsd_committer freebsd_triage 2023-03-27 15:31:54 UTC
I would add .postX to DISTVERSIONSUFFIX and bump PORTREVISION. This is the minimal/simplest change. See also ports b428002358f848c32ccf16a2c2de222ca956689f.
Comment 4 Ting-Wei Lan 2023-03-31 14:55:43 UTC
(In reply to Po-Chuan Hsieh from comment #3)
Did you mean we should prefer dropping the post-release segment to keeping it as a patch level version?

I followed what the handbook said for 'when the version contains letters meaning "patch level"' when finding the solution for the issue.

https://docs.freebsd.org/en/books/porters-handbook/makefiles/#makefile-versions-ex4
Comment 5 Nuno Teixeira freebsd_committer freebsd_triage 2023-04-11 07:29:52 UTC
(In reply to Ting-Wei Lan from comment #4)

I have a similar doubt with "release canditate":

(https://github.com/kopia/kopia/releases/tag/v0.13.0-rc1)

DISTVERSIONPREFIX=      v
DISTVERSION= 0.13.0-rc1

that is translated to:
$ make -VPKGNAME
kopia-0.13.0.rc1

$ pkg version -t 0.13.0 0.13.0.rc1
>

That is correct for this particular case.

(Same result using PORTVERSION)

Any correction on this example?

Thanks
Comment 6 Nuno Teixeira freebsd_committer freebsd_triage 2023-04-11 07:30:57 UTC
(In reply to Nuno Teixeira from comment #5)
(...)

ooops:
kopia-0.13.0.rc1 PORTVERSION 0.13.0-rc1 may not contain '-' '_' or ','.
Comment 7 Nuno Teixeira freebsd_committer freebsd_triage 2023-04-11 07:34:31 UTC
(In reply to Nuno Teixeira from comment #6)
(...)

Error was from using PORTVERSION. No error and no problem fetching distfile from github
Comment 8 Ting-Wei Lan 2023-04-29 08:28:56 UTC
(In reply to Nuno Teixeira from comment #5)
Using DISTVERSION works for pre-releases, but it doesn't work for post-releases.
Comment 9 Yuri Victorovich freebsd_committer freebsd_triage 2023-12-11 17:54:50 UTC
The maintainer's patch is committed with a minor version bump and strip addition.

Thanks!
Comment 10 commit-hook freebsd_committer freebsd_triage 2023-12-11 17:55:14 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=0afb957dcd29b11c77f7d53ebc0da67a9f180589

commit 0afb957dcd29b11c77f7d53ebc0da67a9f180589
Author:     Yuri Victorovich <yuri@FreeBSD.org>
AuthorDate: 2023-12-11 17:52:29 +0000
Commit:     Yuri Victorovich <yuri@FreeBSD.org>
CommitDate: 2023-12-11 17:54:58 +0000

    math/py-pyclipper: update 1.2.1 → 1.3.0p5

    PR:     269501
    Submitted by:   lantw44@gmail.com (maintainer), with minor alteration and addition

 math/py-pyclipper/Makefile | 14 ++++++++++----
 math/py-pyclipper/distinfo |  6 +++---
 2 files changed, 13 insertions(+), 7 deletions(-)