Re: New port: moor

"Lydia Sobot" <chilledfrogs@disroot.org> writes:

> New release came, updated tarball attached, sorry about that

Hi,

sysutils/moor has been added to ports tree. Thanks for your contribution!

While here, I started to look a bit at lang/go module port tree, and
changed it to simplify a bit usage.

The following diff changes sysutils/moor to use TEST_TARGET variable
instead of reimplementing MODGO_TEST_TARGET.

Alternatively, it would be also possible to use TEST_TARGET = ${MODGO_MODNAME}/...
to run tests in *all* directories, but some tests are failing
(TestCodeHighlighting and TestCompressedFiles), so I kept the current
test target.

OK to use TEST_TARGET ?
--
Sebastien Marie


diff --git a/sysutils/moor/Makefile b/sysutils/moor/Makefile
index 2bd6900ca3..cd2d428a89 100644
--- a/sysutils/moor/Makefile
+++ b/sysutils/moor/Makefile
@@ -17,7 +17,7 @@
SUBST_VARS += WRKSRC

MODGO_LDFLAGS += -X main.versionString=${MODGO_VERSION}
-MODGO_TEST_TARGET = cd ${WRKSRC} && ${MODGO_TEST_CMD} ${MODGO_MODNAME}/cmd/moor
+TEST_TARGET = ${MODGO_MODNAME}/cmd/moor

post-patch:
${SUBST_CMD} ${WRKSRC}/internal/reader/reader_test.go