Skip to content

Commit

Permalink
fix bug in msgs cmake
Browse files Browse the repository at this point in the history
- fix a bug in proto path in /msgs for non-GRPC build
- add compiler flag to BoundaryDefense plugin that causes
build failure on some systems
  • Loading branch information
crichardson332 committed Sep 19, 2018
1 parent d9d93df commit f9c54b0
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion msgs/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ foreach(PROTO_FILE ${PROTO_FILE_LIST})
get_filename_component(PROTO_NAME ${PROTO_FILE} NAME_WE)
get_filename_component(ABS_PROTO_NAME ${PROTO_FILE} REALPATH)
get_filename_component(PROTO_PATH ${ABS_PROTO_NAME} DIRECTORY)
get_filename_component(SCRIMMAGE_PROTO_PATH ${SCRIMMAGE_PROTO_PATH} REALPATH)

set(PROTO_FILE_GEN_CPP_FILES
${GEN_DIR}/${PROTO_NAME}.pb.cc
Expand Down Expand Up @@ -54,7 +55,7 @@ foreach(PROTO_FILE ${PROTO_FILE_LIST})
else()
add_custom_command(
OUTPUT ${PROTO_FILE_GEN_FILES}
COMMAND ${PROTOBUF_PROTOC_EXECUTABLE} ${ABS_PROTO_NAME} --proto_path=${PROTO_PATH} --cpp_out=${GEN_DIR} --python_out=${GEN_DIR}
COMMAND ${PROTOBUF_PROTOC_EXECUTABLE} ${ABS_PROTO_NAME} --proto_path=${PROTO_PATH} --proto_path=${SCRIMMAGE_PROTO_PATH} --cpp_out=${GEN_DIR} --python_out=${GEN_DIR}
DEPENDS ${PROTO_FILE}
WORKING_DIRECTORY ${PROTO_PATH}
)
Expand Down
2 changes: 1 addition & 1 deletion src/plugins/autonomy/BoundaryDefense/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ add_library(${LIBRARY_NAME} SHARED
)

target_compile_options(${LIBRARY_NAME}
PRIVATE -Wall -Wno-return-type-c-linkage -std=c++14
PRIVATE -Wall -Wno-return-type-c-linkage -std=c++14 -fext-numeric-literals
)

target_link_libraries(${LIBRARY_NAME}
Expand Down

0 comments on commit f9c54b0

Please sign in to comment.