On Mon, Apr 29, 2024 at 01:45:03PM +0200, Jeremie Courreges-Anglas wrote:
> A few nits:
> - the port doesn't install any doc and DESCR is rather light.  I
>   understand the tool is simple but still, it would be nice to at least
>   have the examples from the README.  Maybe just install that file under
>   share/doc/crawley?
> - make test fails for bad reasons and I don't know how to fix that.
>   NO_TEST=Yes?

Thank you for the useful nits!

Here's a diff that adds both the NO_TEST and README.md to the port.

diff -ur projects/git.disroot.org/mystuff/net/crawley/Makefile 
/usr/ports/mystuff/net/crawley/Makefile
--- projects/git.disroot.org/mystuff/net/crawley/Makefile       Wed May  1 
00:57:52 2024
+++ /usr/ports/mystuff/net/crawley/Makefile     Wed May  1 00:50:43 2024
@@ -14,9 +14,15 @@
 # MIT
 PERMIT_PACKAGE =       Yes
 
+NO_TEST =      Yes
+
 WANTLIB += c pthread
 
 MODULES =      lang/go
+
+post-install:
+       ${INSTALL_DATA_DIR} ${PREFIX}/share/doc/crawley
+       ${INSTALL_DATA} ${WRKSRC}/README.md ${PREFIX}/share/doc/crawley
 
 .include "modules.inc"
 
diff -ur projects/git.disroot.org/mystuff/net/crawley/pkg/PLIST 
/usr/ports/mystuff/net/crawley/pkg/PLIST
--- projects/git.disroot.org/mystuff/net/crawley/pkg/PLIST      Wed May  1 
00:57:52 2024
+++ /usr/ports/mystuff/net/crawley/pkg/PLIST    Wed May  1 00:45:15 2024
@@ -1 +1,3 @@
 @bin bin/crawley
+share/doc/crawley/
+share/doc/crawley/README.md

Reply via email to