Correct gmock/gtest expectations w.r.t. C++11/RTTI

Long story short, some of the tests were failing because they expected either
dynamic_cast or RTTI to be functional and it wasn't.

Move all common CXXFLAGS out to googletest.test.inc.mk and reference it from
googletest.test.mk and .../googletest/Makefile.inc
This commit is contained in:
Enji Cooper
2019-02-20 20:09:59 +00:00
parent 4783c2dda5
commit 833018816d
4 changed files with 18 additions and 5 deletions
+5 -5
View File
@@ -1,12 +1,12 @@
# $FreeBSD$ # $FreeBSD$
CXXFLAGS+= -DGTEST_HAS_POSIX_RE=1 .include <googletest.test.inc.mk>
CXXFLAGS+= -DGTEST_HAS_PTHREAD=1
CXXFLAGS+= -DGTEST_HAS_STREAM_REDIRECTION=1
GTEST_DIR= ${SRCTOP}/contrib/googletest GTEST_DIR= ${SRCTOP}/contrib/googletest
GOOGLEMOCK_SRCROOT= ${GTEST_DIR}/googlemock GOOGLEMOCK_SRCROOT= ${GTEST_DIR}/googlemock
GOOGLETEST_SRCROOT= ${GTEST_DIR}/googletest GOOGLETEST_SRCROOT= ${GTEST_DIR}/googletest
CXXFLAGS+= ${GTESTS_FLAGS}
NO_WTHREAD_SAFETY= NO_WTHREAD_SAFETY=
+1
View File
@@ -73,6 +73,7 @@ FILESDIR= ${BINDIR}/mk
.if ${MK_TESTS} != "no" .if ${MK_TESTS} != "no"
FILES+= atf.test.mk FILES+= atf.test.mk
FILES+= googletest.test.inc.mk
FILES+= googletest.test.mk FILES+= googletest.test.mk
FILES+= plain.test.mk FILES+= plain.test.mk
FILES+= suite.test.mk FILES+= suite.test.mk
+9
View File
@@ -0,0 +1,9 @@
# $FreeBSD$
GTESTS_CXXFLAGS+= -DGTESTS_HAS_RTTI=1
GTESTS_CXXFLAGS+= -DGTESTS_HAS_POSIX_RE=1
GTESTS_CXXFLAGS+= -DGTESTS_HAS_PTHREAD=1
GTESTS_CXXFLAGS+= -DGTESTS_HAS_STREAM_REDIRECTION=1
GTESTS_CXXFLAGS+= -DGTESTS_LANG_CXX11=1
GTESTS_CXXFLAGS+= -frtti
GTESTS_CXXFLAGS+= -std=c++11
+3
View File
@@ -27,10 +27,13 @@
GTESTS?= GTESTS?=
.if !empty(GTESTS) .if !empty(GTESTS)
.include <googletest.test.flags.mk>
PROGS_CXX+= ${GTESTS} PROGS_CXX+= ${GTESTS}
_TESTS+= ${GTESTS} _TESTS+= ${GTESTS}
.for _T in ${GTESTS} .for _T in ${GTESTS}
BINDIR.${_T}= ${TESTSDIR} BINDIR.${_T}= ${TESTSDIR}
CXXFLAGS.${_T}+= ${GTESTS_CXXFLAGS}
MAN.${_T}?= # empty MAN.${_T}?= # empty
SRCS.${_T}?= ${_T}.cc SRCS.${_T}?= ${_T}.cc
TEST_INTERFACE.${_T}= plain TEST_INTERFACE.${_T}= plain