Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 31 Aug 2018 20:14:58 +0000 (UTC)
From:      Jan Beich <jbeich@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r478608 - head/devel/msgpack
Message-ID:  <201808312014.w7VKEwAC012418@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jbeich
Date: Fri Aug 31 20:14:57 2018
New Revision: 478608
URL: https://svnweb.freebsd.org/changeset/ports/478608

Log:
  devel/msgpack: restore TEST_TARGET after r470984
  
  TEST_DEPENDS is pointless without do-test or TEST_TARGET. automake
  required googletest only during "check" but cmake wants it upfront
  during "test". So, move tests into a separate option.

Modified:
  head/devel/msgpack/Makefile   (contents, props changed)

Modified: head/devel/msgpack/Makefile
==============================================================================
--- head/devel/msgpack/Makefile	Fri Aug 31 20:14:40 2018	(r478607)
+++ head/devel/msgpack/Makefile	Fri Aug 31 20:14:57 2018	(r478608)
@@ -11,18 +11,25 @@ COMMENT=	Efficient binary serialization, like JSON but
 LICENSE=	BSL
 LICENSE_FILE=	${WRKSRC}/LICENSE_1_0.txt
 
-TEST_DEPENDS=	googletest>0:devel/googletest
-
 USES=		cmake pathfix
 USE_LDCONFIG=	yes
 USE_GITHUB=	yes
 GH_PROJECT=	msgpack-c
 
+OPTIONS_DEFINE=	TEST
+
+TEST_BUILD_DEPENDS=	googletest>0:devel/googletest
+TEST_TEST_TARGET=	test
+
 .include <bsd.port.options.mk>
 
 .if ${ARCH} == "sparc64"
 # required for __sync _* atomic operations on sparc
 USE_GCC=	yes
 .endif
+
+# poudriere doesn't run tests yet, so hook them manually
+# https://github.com/freebsd/poudriere/pull/355
+pre-install-TEST-on: do-test
 
 .include <bsd.port.mk>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201808312014.w7VKEwAC012418>