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

use underlay workspaces when calculating topological order #590

Merged
merged 1 commit into from Feb 10, 2014

Conversation

Projects
None yet
3 participants
@dirk-thomas
Member

dirk-thomas commented Feb 7, 2014

Depends on ros-infrastructure/catkin_pkg#87 being released as 0.1.26.
Should be backported to Groovy and Hydro.

Note: this patch will significantly increase the time to compute the topological order since all available package.xml files from all underlays must be crawled and parsed.

@tfoote @wjwwood Please review.

@tfoote

This comment has been minimized.

Member

tfoote commented Feb 7, 2014

+1

1 similar comment
@wjwwood

This comment has been minimized.

Member

wjwwood commented Feb 7, 2014

+1

dirk-thomas added a commit that referenced this pull request Feb 10, 2014

Merge pull request #590 from ros/topological_order_with_underlays
use underlay workspaces when calculating topological order

@dirk-thomas dirk-thomas merged commit 72141df into indigo-devel Feb 10, 2014

@dirk-thomas dirk-thomas deleted the topological_order_with_underlays branch Feb 10, 2014

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