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

Parts of this require C++11 #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

chhtz
Copy link

@chhtz chhtz commented Jul 9, 2018

No description provided.

@saarnold
Copy link
Owner

saarnold commented Jul 9, 2018

I guess C++11 code was introduced with the use of the explicit specifier in the PoseWithCovariance constructor.
This PR should be applied on the main repository: https://github.com/rock-core/base-types
Where we can decide to either merge it or remove the explicit specifiers.

@chhtz
Copy link
Author

chhtz commented Jul 9, 2018

This may actually be caused by a local hack by me. I'm currently essentially including a (or referring to) a local Rock workspace. Therefore, Vizkit3d is found, which apparently requires C++11, e.g., there is a std::shared_ptr<ClickHandler> in Vizkit3DPlugin.hpp.

I'll try again with a "rock-free" ROS build.

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

Successfully merging this pull request may close these issues.

2 participants