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.

Sign up for GitHub

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

Jump to bottom

Convert CMake tests to <test_depend> #157

Closed
davetcoleman opened this issue Apr 17, 2014 · 3 comments
Closed

Convert CMake tests to <test_depend> #157

davetcoleman opened this issue Apr 17, 2014 · 3 comments

Comments

@davetcoleman
Copy link
Member

http://wiki.ros.org/catkin/package.xml#Build.2C_Run.2C_and_Test_Dependencies

Me:

Shouldn't this be <test_depend> ?

Adolfo:

It actually should, yes. At least in Indigo since the merge of ros/catkin#612. This code was written for Hydro, and by that time <test_depends> were being ignored by the buildfarm. My suggestion would be to merge the PR first, and then do a pass across the whole of ros_control to fix build/test dependencies.
@davetcoleman davetcoleman mentioned this issue Apr 17, 2014
Merged
@davetcoleman
Copy link
Member Author

Cleaning up issues.

@adolfo-rt
Copy link
Member

This issue, although somewhat old, has in fact not been addressed. I would keep it open as a reminder of pending tasks.

@adolfo-rt adolfo-rt reopened this Oct 14, 2014
@matthew-reynolds matthew-reynolds mentioned this issue Jan 12, 2020
Merged
@bmagyar
Copy link
Member

bmagyar commented Jan 12, 2020

This has been long resolved.

@bmagyar bmagyar closed this as completed Jan 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@davetcoleman @adolfo-rt @bmagyar