jhu-lcsr-forks / rtt_ros_integration

Orocos-ROS integration libraries and tools
1 stars 2 forks source link

Fixed recursive import of ROS packages #46

Closed meyerj closed 10 years ago

meyerj commented 10 years ago

Without this patch the ros.import("foo") operation returned an error if package foo has already been loaded. Even if the same package is not imported twice explicitly, it might happen that another package that depends on foo has already been imported.

Example:

Deployer [S]> import("rtt_ros")
 = true                

Deployer [S]> ros.import("rtt_sensor_msgs")
 = true                

Deployer [S]> ros.import("rtt_geometry_msgs")
23.826 [ Warning][ROSService::import("rtt_geometry_msgs")] Could not load any plugins from ROS package "rtt_geometry_msgs" or it's dependencies.
 = false               
jbohren commented 10 years ago

Good catch.