Created attachment 185795 [details] svn diff dns/void-zones-tools comes with very little documentation. This patch adds a pkg-message explaining how to get started, and installs the README.md file in ${DOCSDIR}. Also: fixes the problem of the /usr/local/etc/my_void_hosts.txt template not getting created[1] by adding a new check to void-zones-update.sh. [1] void-zones-update.sh checks if the dir /usr/local/etc/void-zones-tools exists, and if not, it creates it and installs the my_void_hosts.txt template into it. Since the port creates that dir during install, the shell script will never create the template.
Created attachment 185816 [details] Proposed additions to maintainer's patch Hi, What do you think about the following? I just tweaked your patch so that instead of referring to ${PREFIX} in pkg-message it substitutes in the actual value of PREFIX the port is built with. This is the usual practice in the ports. Otherwise this all looks good.
(In reply to Matthew Seaman from comment #1) That does indeed look nicer, and I learned something new today (I was looking at some other ports to find out if/how such a substitution could be done, but didn't find anything).
Great. Commmitted, thanks!
A commit references this bug: Author: matthew Date: Sun Aug 27 15:08:57 UTC 2017 New revision: 448825 URL: https://svnweb.freebsd.org/changeset/ports/448825 Log: Add additional documentation and a pkg-message with further hints. PR: 221841 Submitted by: vidar@karlsen.tech (maintainer) Changes: head/dns/void-zones-tools/Makefile head/dns/void-zones-tools/files/ head/dns/void-zones-tools/files/patch-void-zones-update.sh head/dns/void-zones-tools/files/pkg-message.in head/dns/void-zones-tools/pkg-plist