Skip to content

Comments

ros_overview: update depend tag#162

Open
s-a-tanjim wants to merge 4 commits intoosrf:masterfrom
s-a-tanjim:patch-1
Open

ros_overview: update depend tag#162
s-a-tanjim wants to merge 4 commits intoosrf:masterfrom
s-a-tanjim:patch-1

Conversation

@s-a-tanjim
Copy link

@mayman99
Copy link
Contributor

mayman99 commented Jun 3, 2022

Thank you for catching this!
But according to this http://docs.ros.org/en/jade/api/catkin/html/howto/format2/migrating_from_format_1.html , shouldn't we add <build_export_depend>foo</build_export_depend> as well?

@s-a-tanjim
Copy link
Author

Oww my bad! Actually my program compiled fine without <build_export_depend>foo</build_export_depend>.
But I think adding <depend>foo</depend> is better option. Thanks anyway!

@scpeters scpeters changed the title Update tutorial.md ros_overview: update depend tag Mar 6, 2024
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