kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danburk...@apache.org
Subject [1/3] kudu git commit: [fs] fix build on OS X
Date Fri, 28 Apr 2017 22:07:53 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 7b582a896 -> fdc022fe6


[fs] fix build on OS X

Added kudu_fs dependency for kudu_fs_test_util library.

Prior to this patch, the following error was reported if trying to build
on OS X:

Undefined symbols for architecture x86_64:
  "kudu::fs::LogBlockManager::kContainerDataFileSuffix", referenced from:
      kudu::fs::LBMCorruptor::Init() in log_block_manager-test-util.cc.o
      kudu::fs::LBMCorruptor::CreateIncompleteContainer()
          in log_block_manager-test-util.cc.o
  "kudu::fs::LogBlockManager::kContainerMetadataFileSuffix", referenced from:
      kudu::fs::LBMCorruptor::Init() in log_block_manager-test-util.cc.o
      kudu::fs::LBMCorruptor::CreateIncompleteContainer()
          in log_block_manager-test-util.cc.o
  "fLU64::FLAGS_log_container_max_size", referenced from:
      kudu::fs::LBMCorruptor::Init() in log_block_manager-test-util.cc.o
  "kudu::BlockId::CopyToPB(kudu::BlockIdPB*) const", referenced from:
      kudu::fs::LBMCorruptor::AddMalformedRecordToContainer()
          in log_block_manager-test-util.cc.o
      kudu::fs::LBMCorruptor::AddMisalignedBlockToContainer()
          in log_block_manager-test-util.cc.o
      kudu::fs::LBMCorruptor::AddPartialRecordToContainer()
          in log_block_manager-test-util.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/libkudu_fs_test_util.dylib] Error 1

Change-Id: I60e16871818bb8994a2ecc59628ba22633ece3ba
Reviewed-on: http://gerrit.cloudera.org:8080/6756
Reviewed-by: Adar Dembo <adar@cloudera.com>
Tested-by: Alexey Serbin <aserbin@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/31751df1
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/31751df1
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/31751df1

Branch: refs/heads/master
Commit: 31751df1ee8d05f620ff6dbf73e70fd7fd09a467
Parents: 7b582a8
Author: Alexey Serbin <aserbin@cloudera.com>
Authored: Thu Apr 27 22:32:47 2017 -0700
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Fri Apr 28 06:05:36 2017 +0000

----------------------------------------------------------------------
 src/kudu/fs/CMakeLists.txt | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/31751df1/src/kudu/fs/CMakeLists.txt
----------------------------------------------------------------------
diff --git a/src/kudu/fs/CMakeLists.txt b/src/kudu/fs/CMakeLists.txt
index ffe8066..78de838 100644
--- a/src/kudu/fs/CMakeLists.txt
+++ b/src/kudu/fs/CMakeLists.txt
@@ -46,6 +46,7 @@ add_library(kudu_fs_test_util
 
 target_link_libraries(kudu_fs_test_util
   fs_proto
+  kudu_fs
   kudu_util
   gutil)
 


Mime
View raw message