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

Find catkin before invoking catkin_package() #9

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

spmaniato
Copy link

@spmaniato spmaniato commented May 18, 2016

Some fixes to the dubins traffic ROS packages' CMakeLists.txt files. I think you've been depending on other packages finding catkin, which catkin_make allows. With these fixes, you can also build with catkin (aka catkin_tools)

I'm not as confident about the dubins_traffic_utils change because there's some extra logic there.

@slivingston slivingston self-assigned this May 21, 2016
@slivingston
Copy link
Member

My plan is to merge the code that was used for evaluation at ICRA 2016 and then to consider this pull request.

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