Merge pull request #49 from pfnet-research/add-protobuf-version-check

add protobuf version check
pfnet-research · Jul 19, 2018 · ff8631a · ff8631a
2 parents 45653bb + 8963580
commit ff8631a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -40,7 +40,7 @@ set(EXTERNAL_DIR ${CMAKE_SOURCE_DIR}/external)
 set(ONNX_DIR ${EXTERNAL_DIR}/onnx)
 set(Protobuf_PROTOC_EXECUTABLE protoc CACHE STRING "protoc path")
 include(FindProtobuf)
-find_package(Protobuf REQUIRED)
+find_package(Protobuf 2.6.1 REQUIRED)
 include_directories(${PROTOBUF_INCLUDE_DIR})
 execute_process(COMMAND git submodule update --init -- ${ONNX_DIR} WORKING_DIRECTORY ${CMAKE_SOURCE_DIR})
 execute_process(COMMAND ${Protobuf_PROTOC_EXECUTABLE} -I=${ONNX_DIR} --cpp_out=${ONNX_DIR} ${ONNX_DIR}/onnx/onnx.proto WORKING_DIRECTORY ${CMAKE_SOURCE_DIR})

diff --git a/README.md b/README.md
@@ -42,7 +42,7 @@ This codebase contains C API and C++ API.
 ## Requirements
 
 - MKL-DNN Library (0.14 or later)
-- Protocol Buffers (3.5.1 is checked)
+- Protocol Buffers (2.6.1 or later)
 
 ## Build