search

JeroenDM / moveit_opw_kinematics_plugin

An attempt at writing a MoveIt! plugin for opw_kinematics.
http://wiki.ros.org/moveit_opw_kinematics_plugin
Apache License 2.0
7 stars 7 forks source link

specify C++14 in a new more cross-platform way #58

Closed JeroenDM closed 4 years ago

JeroenDM commented 4 years ago

This change matches how the C++ standard is specified in moveit_kinematics, as discussed in #48.

set(CMAKE_CXX_STANDARD 14)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)

Ideally, I would like to test this on an msvc compiler, but I did not look into it yet. Maybe some of the dependencies are not supported yet.

simonschmeisser commented 4 years ago

I guess someone from M$ will show up shortly once we add this to MoveIt tutorials and fix anything remaining, no need to be proactive ;)