Bug 270539 - net/binkd: update to 1.1a-115
Summary: net/binkd: update to 1.1a-115
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: Fernando Apesteguía
URL: https://github.com/pgul/binkd/blob/ma...
Keywords: needs-qa
Depends on:
Blocks:
 
Reported: 2023-03-30 10:30 UTC by Andriy Dzedolik
Modified: 2023-04-03 07:20 UTC (History)
3 users (show)

See Also:


Attachments
Makefile and distinfo patch (2.04 KB, patch)
2023-03-30 10:30 UTC, Andriy Dzedolik
no flags Details | Diff
Fixed and linted patch (2.75 KB, patch)
2023-03-31 09:13 UTC, Andriy Dzedolik
no flags Details | Diff
net/binkd patch (2.73 KB, patch)
2023-03-31 12:21 UTC, Andriy Dzedolik
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andriy Dzedolik 2023-03-30 10:30:36 UTC
Created attachment 241206 [details]
Makefile and distinfo patch

Updated the Makefile and distinfo to use GitHub revision of binkd 1.1a-115, which is the latest formally released version.
Updated Makefile with some new compile options.
Updated Makefile to reflect the documentation files changes.
Comment 1 Graham Perrin freebsd_committer freebsd_triage 2023-03-31 01:35:49 UTC
Can you link to release notes? 

Thanks

Incidentally, I get 502 Bad Gateway for <http://binkd.grumbler.org/> at the time of writing.
Comment 2 Fernando Apesteguía freebsd_committer freebsd_triage 2023-03-31 07:07:24 UTC
Thanks for your submission. Some issues.

Q/A: PORTREVISION should be removed

Specify PORTVERSION or DISTVERSION, but not both.

Please, use portlint (ports-mgmt/portlint). It is a great tool :-)
                                        
                                        
Thanks!
Comment 3 Andriy Dzedolik 2023-03-31 07:51:45 UTC
(In reply to Graham Perrin from comment #1)
Would this work as a 'rel.notes' https://github.com/pgul/binkd/blob/master/HISTORY ?

Thanks!
Comment 4 Andriy Dzedolik 2023-03-31 07:55:12 UTC
(In reply to Fernando Apesteguía from comment #2)

Sure, will do. I've raised this ticket to see whether we can do anything at all, as binkd was quite outdated for last 3 or 4 FreeBSD major releases ...
I'm going to add some configure options and fix documentation/examples, then I'll re-submit the patch.

Thanks!
Comment 5 Fernando Apesteguía freebsd_committer freebsd_triage 2023-03-31 08:13:10 UTC
(In reply to dz from comment #3)
Yes, thanks
Comment 6 Andriy Dzedolik 2023-03-31 09:13:18 UTC
Created attachment 241224 [details]
Fixed and linted patch

Please check attached patch.
Comment 7 Fernando Apesteguía freebsd_committer freebsd_triage 2023-03-31 11:45:52 UTC
====> Running Q/A tests (stage-qa)
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
Error: Missing: etc/rc.d/binkd
===> Error: Plist issues found.
*** Error code 1

Hi Andryi,

Please, test patches before submission. Use poudriere or another type of clean environment.
Comment 8 Andriy Dzedolik 2023-03-31 12:21:30 UTC
(In reply to Fernando Apesteguía from comment #7)

I've tested it with make package but missed to notice warning message, rc.d scripts was referred incorrectly - attaching fixed patch.
Comment 9 Andriy Dzedolik 2023-03-31 12:21:58 UTC
Created attachment 241228 [details]
net/binkd patch
Comment 10 Fernando Apesteguía freebsd_committer freebsd_triage 2023-04-03 07:19:53 UTC
Committed,

Thanks!
Comment 11 commit-hook freebsd_committer freebsd_triage 2023-04-03 07:19:56 UTC
A commit in branch main references this bug:

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

commit bf36dd0be21aa644d56e6767129ff731371d4b39
Author:     Andriy Dzedolik <dz@dolik.dev>
AuthorDate: 2023-04-01 10:53:52 +0000
Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
CommitDate: 2023-04-03 07:12:36 +0000

    net/binkd: update to 1.1a-115

    ChangeLog: https://github.com/pgul/binkd/blob/master/HISTORY

    PR:             270539
    Reported by:    dz@dolik.dev

 net/binkd/Makefile  | 41 +++++++++++++++++++++++------------------
 net/binkd/distinfo  |  5 +++--
 net/binkd/pkg-plist | 10 +++++-----
 3 files changed, 31 insertions(+), 25 deletions(-)