Bug 254765

Summary: dns/knot-resolver fails packaging
Product: Ports & Packages Reporter: Marcel <marcel>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Closed FIXED    
Severity: Affects Only Me CC: freebsd, kbowling
Priority: --- Flags: bugzilla: maintainer-feedback? (freebsd)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
formatting and adjustments of knot-resolver freebsd: maintainer-approval+

Description Marcel 2021-04-04 15:41:59 UTC 
Port fails packaging:

===>  Building package for knot-resolver-5.3.0
@sample with 1 single argument expect ".sample" extension
pkg-static: Fail to apply keyword 'sample'
*** Error code 1

Issue seems to be in the first line of the port pkg-plist:

@sample %%ETCDIR%%/kresd.conf

where no ".sample" conf file is provided

Seems like it was added as part of ports rP542054.
Comment 1 commit-hook freebsd_committer freebsd_triage 2021-04-13 23:10:44 UTC 
A commit in branch main references this bug:

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

commit d94ae2c3c666fd3c26e4a5e572ad084d88a8b5f3
Author:     Kevin Bowling <kbowling@FreeBSD.org>
AuthorDate: 2021-04-13 23:07:49 +0000
Commit:     Kevin Bowling <kbowling@FreeBSD.org>
CommitDate: 2021-04-13 23:07:49 +0000

    dns/knot-resolver: Fix @sample use

    The port was not using the @sample functionality correctly, stage the
    correct sample config in post-install and fix the pkg-plist.

    PR:             254765
    Reported by:    Marcel <marcel@brickporch.com>
    Approved by:    maintainer timeout

 dns/knot-resolver/Makefile  | 4 +++-
 dns/knot-resolver/pkg-plist | 2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)
Comment 2 Kevin Bowling freebsd_committer freebsd_triage 2021-04-13 23:11:11 UTC 
Thanks for the report!
Comment 3 Leo Vandewoestijne 2021-04-13 23:14:29 UTC 
Created attachment 224093 [details]
formatting and adjustments of knot-resolver

As per email of Baptiste Daroussin:
added the post-install command,
and renamed the sample in pkg-plist.

The rest is cosmetics, that pkgfmt was complaining about.

I did not add PORTREVISION.

Tested succesful with poudriere.
Comment 4 commit-hook freebsd_committer freebsd_triage 2021-04-14 00:40:01 UTC 
A commit in branch main references this bug:

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

commit fed0c40be92a3890a800250b21c5d470f1aad294
Author:     Kevin Bowling <kbowling@FreeBSD.org>
AuthorDate: 2021-04-14 00:32:57 +0000
Commit:     Kevin Bowling <kbowling@FreeBSD.org>
CommitDate: 2021-04-14 00:39:05 +0000

    dns/knot-resolver: Apply maintainer's cleanup

    This also fixes the kresd.conf @sample, prefer the maintainer's
    approach in the cleanup patch that arrived after d94ae2c3c666

    PR:             254765
    Approved by:    Leo Vandewoestijne <freebsd@dns.company> (maintainer)

 dns/knot-resolver/Makefile | 23 +++++++++++------------
 1 file changed, 11 insertions(+), 12 deletions(-)
Comment 5 Kevin Bowling freebsd_committer freebsd_triage 2021-04-14 00:41:00 UTC 
(In reply to Leo Vandewoestijne from comment #3)
Thanks, I've applied your changes as seen in the commit-hook.  Please consider using git in the future, it will make applying patches easier.