Andrew Wong has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16038 )

Change subject: Fix linker error with tablet_replica-test-base.cc on Mac OS
......................................................................

Fix linker error with tablet_replica-test-base.cc on Mac OS

Hit following linker error on Mac OS 10.15.4 which indicates
missing gtest related library.

Undefined symbols for architecture x86_64:
  "kudu::KuduTest::SetUp()", referenced from:
      kudu::tablet::KuduTabletTest::SetUp() in tablet_replica-test-base.cc.o
  "testing::Test::TearDown()", referenced from:
      kudu::tablet::TabletReplicaTestBase::TearDown() in 
tablet_replica-test-base.cc.o
  "testing::Message::Message()", referenced from:
      kudu::tablet::TabletReplicaTestBase::SetUp() in 
tablet_replica-test-base.cc.o
  "testing::internal::AssertHelper::AssertHelper(testing::TestPartResult::Type, 
char const*, int, char const*)", referenced from:
      kudu::tablet::TabletReplicaTestBase::SetUp() in 
tablet_replica-test-base.cc.o
  "testing::internal::AssertHelper::~AssertHelper()", referenced from:
      kudu::tablet::TabletReplicaTestBase::SetUp() in 
tablet_replica-test-base.cc.o
  "testing::internal::AssertHelper::operator=(testing::Message const&) const", 
referenced from:
      kudu::tablet::TabletReplicaTestBase::SetUp() in 
tablet_replica-test-base.cc.o
  "typeinfo for kudu::KuduTest", referenced from:
      typeinfo for kudu::tablet::KuduTabletTest in tablet_replica-test-base.cc.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [lib/libtablet_test_util.dylib] Error 1

Added kudu_test_util as suggested by Andrew to fix the linker issue.

Change-Id: Ie5857cc0b3436452b5de5aa85fbffe42fad7dff3
Reviewed-on: http://gerrit.cloudera.org:8080/16038
Tested-by: Kudu Jenkins
Reviewed-by: Andrew Wong <[email protected]>
---
M src/kudu/tablet/CMakeLists.txt
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Andrew Wong: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16038
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie5857cc0b3436452b5de5aa85fbffe42fad7dff3
Gerrit-Change-Number: 16038
Gerrit-PatchSet: 2
Gerrit-Owner: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to