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

Simplify dependency declarations #46

Closed

bmagyar opened this issue Feb 4, 2020 · 1 comment

Assignees

Member

bmagyar commented Feb 4, 2020

Specifically I mean to do build_depend + exec_depend = depend.
Should you find anything else you could simplify/clean up, please go ahead.

The text was updated successfully, but these errors were encountered:

bmagyar assigned jordan-palacios

bmagyar mentioned this issue

Use pluginlib instead of class_loader for loading controllers #41

Merged

2 tasks

Member

jordan-palacios commented Feb 10, 2020 •

edited

Loading

Keep in mind that in package.xml format 2:
<depend> = <build_depend> + <build_export_depend> + <exec_depend>

More information here

I could not find any instances in our current packages where all three depends are being used and a <build> would be suitable.

bmagyar closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment