Skip to content

Commit

Permalink
suppress xacro warnings when determining dependencies (#182)
Browse files Browse the repository at this point in the history
Instead of using xacro's stderr, use the process result to determine success.
On stderr all kinds of other warnings are published as well.
  • Loading branch information
rhaschke committed Mar 27, 2018
1 parent b2f9ff3 commit 3ac7377
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions cmake/xacro-extras.cmake.em
Original file line number Diff line number Diff line change
Expand Up @@ -86,13 +86,14 @@ function(xacro_add_xacro_file input)
message(STATUS "xacro: determining deps for: " ${input} " ...")
execute_process(COMMAND ${CATKIN_ENV} ${_xacro_py} ${_XACRO_INORDER} --deps ${input} ${_XACRO_REMAP}
WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
RESULT_VARIABLE _xacro_result
ERROR_VARIABLE _xacro_err
OUTPUT_VARIABLE _xacro_deps_result
OUTPUT_STRIP_TRAILING_WHITESPACE)
if(_xacro_err)
if(_xacro_result)
message(WARNING "failed to determine deps for: ${input}
${_xacro_err}")
endif(_xacro_err)
endif(_xacro_result)

separate_arguments(_xacro_deps_result)

Expand Down

0 comments on commit 3ac7377

Please sign in to comment.