Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

All outputs from catkin build are stdOutputs even when errors. #419

Closed
Levi-Armstrong opened this issue Dec 30, 2016 · 3 comments
Closed

Comments

@Levi-Armstrong
Copy link

System Info

Build / Run Issue

I am currently working on integrating catkin_tools into the ROS Project Manager for Qt Creator and I noticed that it appears that all of the build outputs are going out as stdOuput where when using catkin_make warning like the one below go out as stdError. This is causing issues with Qt Creator because it expects certain compiler warnings and errors to go out as stdErrror.

In file included from /opt/ros/indigo/include/ros/ros.h:40:0,
                 from /opt/ros/indigo/include/simple_message/log_wrapper.h:36,
                 from /home/larmstrong/catkin_ws/catkin_rss_ws/src/motoman/motoman_driver/include/motoman_driver/simple_message/motoman_motion_ctrl.h:38,
                 from /home/larmstrong/catkin_ws/catkin_rss_ws/src/motoman/motoman_driver/include/motoman_driver/motion_ctrl.h:36,
                 from /home/larmstrong/catkin_ws/catkin_rss_ws/src/motoman/motoman_driver/include/motoman_driver/joint_trajectory_streamer.h:38,
                 from /home/larmstrong/catkin_ws/catkin_rss_ws/src/motoman/motoman_driver/src/joint_trajectory_streamer.cpp:32:
/home/larmstrong/catkin_ws/catkin_rss_ws/src/motoman/motoman_driver/src/joint_trajectory_streamer.cpp: In static member function ‘static bool motoman::joint_trajectory_streamer::MotomanJointTrajectoryStreamer::VectorToJointData(const std::vector<double>&, industrial::joint_data::JointData&)’:
/opt/ros/indigo/include/ros/console.h:342:176: warning: format ‘%d’ expects argument of type ‘int’, but argument 8 has type ‘std::vector<double>::size_type {aka long unsigned int}’ [-Wformat=]
     ::ros::console::print(filter, __rosconsole_define_location__loc.logger_, __rosconsole_define_location__loc.level_, __FILE__, __LINE__, __ROSCONSOLE_FUNCTION__, __VA_ARGS__)
                                                                                                                                                                                ^
/opt/ros/indigo/include/ros/console.h:345:5: note: in expansion of macro ‘ROSCONSOLE_PRINT_AT_LOCATION_WITH_FILTER’
     ROSCONSOLE_PRINT_AT_LOCATION_WITH_FILTER(0, __VA_ARGS__)
     ^
/opt/ros/indigo/include/ros/console.h:375:7: note: in expansion of macro ‘ROSCONSOLE_PRINT_AT_LOCATION’
       ROSCONSOLE_PRINT_AT_LOCATION(__VA_ARGS__); \
       ^
/opt/ros/indigo/include/ros/console.h:557:35: note: in expansion of macro ‘ROS_LOG_COND’
 #define ROS_LOG(level, name, ...) ROS_LOG_COND(true, level, name, __VA_ARGS__)
                                   ^
/opt/ros/indigo/include/rosconsole/macros_generated.h:214:24: note: in expansion of macro ‘ROS_LOG’
 #define ROS_ERROR(...) ROS_LOG(::ros::console::levels::Error, ROSCONSOLE_DEFAULT_NAME, __VA_ARGS__)
                        ^
/home/larmstrong/catkin_ws/catkin_rss_ws/src/motoman/motoman_driver/src/joint_trajectory_streamer.cpp:65:39: note: in expansion of macro ‘ROS_ERROR’
 #define ROS_ERROR_RETURN(rtn,...) do {ROS_ERROR(__VA_ARGS__); return(rtn);} while(0)
                                       ^
/home/larmstrong/catkin_ws/catkin_rss_ws/src/motoman/motoman_driver/src/joint_trajectory_streamer.cpp:304:5: note: in expansion of macro ‘ROS_ERROR_RETURN’
     ROS_ERROR_RETURN(false, "Failed to copy to JointData.  Len (%d) out of range (0 to %d)",
     ^

Expected Behavior

I would have expected that stdErrors comming from cmake and make would pass through as stdErrors with catkin_tools.

Actual Behavior

It appears that all outputs from catkin build are stdOutput.

@Levi-Armstrong
Copy link
Author

It looks like #424 addressed this issue.

@wjwwood
Copy link
Member

wjwwood commented Jan 27, 2017

Ok, I'll close it for now then.

@wjwwood wjwwood closed this as completed Jan 27, 2017
@Levi-Armstrong
Copy link
Author

Thank you for making this change, I just tested the latest version and everything works great using catkin tools within an IDE!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants